kernel32 tests: fix memory leaks (found by Smatch).

Lionel Debroux lionel_debroux at yahoo.fr
Sat Oct 20 14:28:21 CDT 2007


---
 dlls/kernel32/tests/pipe.c |    7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/dlls/kernel32/tests/pipe.c b/dlls/kernel32/tests/pipe.c
index 9150e50..ce0f5b6 100644
--- a/dlls/kernel32/tests/pipe.c
+++ b/dlls/kernel32/tests/pipe.c
@@ -805,6 +805,7 @@ static void test_CreatePipe(void)
     /* But now we need to get informed that the pipe is closed */
     ok(ReadFile(piperead,readbuf,sizeof(readbuf),&read, NULL) == 0, "Broken pipe not detected\n");
     ok(CloseHandle(piperead), "CloseHandle for the read pipe failed\n");
+    HeapFree(GetProcessHeap(), 0, buffer);
 }
 
 struct named_pipe_client_params
@@ -1022,7 +1023,11 @@ static BOOL are_all_privileges_disabled(HANDLE hToken)
     {
         Privileges = HeapAlloc(GetProcessHeap(), 0, Size);
         ret = GetTokenInformation(hToken, TokenPrivileges, Privileges, Size, &Size);
-        if (!ret) return FALSE;
+        if (!ret)
+        {
+            HeapFree(GetProcessHeap(), 0, Privileges);
+            return FALSE;
+        }
     }
     else
         return FALSE;
-- 
1.5.3.2


--------------030200030902070906030802--



More information about the wine-patches mailing list