Alexandre Julliard : advapi32/tests: Avoid accessing an uninitialized pointer.

Alexandre Julliard julliard at winehq.org
Tue Jun 16 09:09:34 CDT 2009


Module: wine
Branch: master
Commit: ac5002ed2b7a475980ac3ba60fa24320d42d2e70
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ac5002ed2b7a475980ac3ba60fa24320d42d2e70

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Tue Jun 16 13:27:32 2009 +0200

advapi32/tests: Avoid accessing an uninitialized pointer.

---

 dlls/advapi32/tests/security.c |    9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c
index b68629d..af52a3a 100644
--- a/dlls/advapi32/tests/security.c
+++ b/dlls/advapi32/tests/security.c
@@ -1237,10 +1237,11 @@ static void test_token_attr(void)
         Domain[0] = '\0';
         ret = LookupAccountSid(NULL, Groups->Groups[i].Sid, Name, &NameLength, Domain, &DomainLength, &SidNameUse);
         if (ret)
-            trace("\t%s, %s\\%s use: %d attr: 0x%08x\n", SidString, Domain, Name, SidNameUse, Groups->Groups[i].Attributes);
-        else
-            trace("\t%s, attr: 0x%08x LookupAccountSid failed with error %d\n", SidString, Groups->Groups[i].Attributes, GetLastError());
-        LocalFree(SidString);
+        {
+            trace("%s, %s\\%s use: %d attr: 0x%08x\n", SidString, Domain, Name, SidNameUse, Groups->Groups[i].Attributes);
+            LocalFree(SidString);
+        }
+        else trace("attr: 0x%08x LookupAccountSid failed with error %d\n", Groups->Groups[i].Attributes, GetLastError());
     }
     HeapFree(GetProcessHeap(), 0, Groups);
 




More information about the wine-cvs mailing list