[PATCH 0/1] MR1658: advapi32/tests: Only free GetSecurityInfo pointer on success.

[Rémi Bernon]

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/1658

[PATCH 1/1] advapi32/tests: Only free GetSecurityInfo pointer on success.

[Rémi Bernon]

From: Rémi Bernon <rbernon(a)codeweavers.com>

---
 dlls/advapi32/tests/security.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c
index 51bf60d0f36..8b4a868ee11 100644
--- a/dlls/advapi32/tests/security.c
+++ b/dlls/advapi32/tests/security.c
@@ -8083,7 +8083,7 @@ static void test_pseudo_handle_security(void)
                     "key %p: got error %u\n", keys[i], ret);
         else
             ok(!ret, "key %p: got error %u\n", keys[i], ret);
-        LocalFree(sd_ptr);
+        if (!ret) LocalFree(sd_ptr);
 
         ret = GetSecurityInfo(keys[i], SE_KERNEL_OBJECT,
                 DACL_SECURITY_INFORMATION, NULL, NULL, NULL, NULL, &sd_ptr);
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/1658