[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;