Matteo Bruni : advapi32/tests: Fix a couple of leaks (Valgrind).

Alexandre Julliard julliard at winehq.org
Thu Jan 11 15:44:32 CST 2018


Module: wine
Branch: master
Commit: 9ccce0ce645f089063ed61bec53644a138ac0a37
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=9ccce0ce645f089063ed61bec53644a138ac0a37

Author: Matteo Bruni <mbruni at codeweavers.com>
Date:   Wed Jan 10 23:44:25 2018 +0100

advapi32/tests: Fix a couple of leaks (Valgrind).

Signed-off-by: Matteo Bruni <mbruni at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/advapi32/tests/security.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c
index a710174..57889dc 100644
--- a/dlls/advapi32/tests/security.c
+++ b/dlls/advapi32/tests/security.c
@@ -6571,6 +6571,8 @@ static void test_system_security_access(void)
     /* privilege is checked on access */
     err = GetSecurityInfo( hkey, SE_REGISTRY_KEY, SACL_SECURITY_INFORMATION, NULL, NULL, NULL, &sacl, &sd );
     todo_wine ok( err == ERROR_PRIVILEGE_NOT_HELD, "got %u\n", err );
+    if (err == ERROR_SUCCESS)
+        LocalFree( sd );
 
     priv.PrivilegeCount = 1;
     priv.Privileges[0].Luid = luid;
@@ -7085,6 +7087,7 @@ static void test_token_security_descriptor(void)
     CloseHandle(info.hThread);
 
     LocalFree(acl_child);
+    HeapFree(GetProcessHeap(), 0, sd2);
     LocalFree(psid);
 
     CloseHandle(token3);




More information about the wine-cvs mailing list