[tfaber] 74417: [EVENTVWR] - Make user name caching actually work CORE-12798

Author: tfaber
Date: Thu Apr 27 00:17:08 2017
New Revision: 74417

URL: http://svn.reactos.org/svn/reactos?rev=74417&view=rev
Log:
[EVENTVWR]
- Make user name caching actually work
CORE-12798

Modified:
    trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c

Modified: trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mscutils/...
==============================================================================
--- trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c	[iso-8859-1] (original)
+++ trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c	[iso-8859-1] Thu Apr 27 00:17:08 2017
@@ -1261,7 +1261,7 @@

BOOL
 GetEventUserName(IN PEVENTLOGRECORD pelr,
-                 IN OUT PSID pLastSid,
+                 IN OUT PSID *pLastSid,
                  OUT PWCHAR pszUser) // TODO: Add IN DWORD BufLen
 {
     PSID pCurrentSid;
@@ -1278,10 +1278,10 @@

if (!IsValidSid(pCurrentSid))
     {
-        pLastSid = NULL;
+        *pLastSid = NULL;
         return FALSE;
     }
-    else if (pLastSid && EqualSid(pLastSid, pCurrentSid))
+    else if (*pLastSid && EqualSid(*pLastSid, pCurrentSid))
     {
         return TRUE;
     }
@@ -1325,7 +1325,7 @@
         }
     }

-    pLastSid = Success ? pCurrentSid : NULL;
+    *pLastSid = Success ? pCurrentSid : NULL;

return Success;
 }
@@ -1604,7 +1604,7 @@
             GetTimeFormatW(LOCALE_USER_DEFAULT, 0, &time, NULL, szLocalTime, ARRAYSIZE(szLocalTime));

/* Get the username that generated the event, and filter it */
-            lpszUsername = GetEventUserName(pEvlrTmp, pLastSid, szUsername) ? szUsername : szNoUsername;
+            lpszUsername = GetEventUserName(pEvlrTmp, &pLastSid, szUsername) ? szUsername : szNoUsername;

if (!FilterByString(EventLogFilter->Users, lpszUsername))
                 goto SkipEvent;