Paul Vriens : kernel32/tests: Prevent memory leak.

Alexandre Julliard julliard at wine.codeweavers.com
Mon Jan 8 14:44:34 CST 2007


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

Author: Paul Vriens <paul.vriens.wine at gmail.com>
Date:   Mon Jan  8 19:03:10 2007 +0100

kernel32/tests: Prevent memory leak.

---

 dlls/kernel32/tests/virtual.c |   11 ++++++-----
 1 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/dlls/kernel32/tests/virtual.c b/dlls/kernel32/tests/virtual.c
index 30b037a..767008f 100644
--- a/dlls/kernel32/tests/virtual.c
+++ b/dlls/kernel32/tests/virtual.c
@@ -58,11 +58,6 @@ static void test_VirtualAllocEx(void)
     hProcess = create_target_process("sleep");
     ok(hProcess != NULL, "Can't start process\n");
 
-    src = (char *) HeapAlloc( GetProcessHeap(), 0, alloc_size );
-    dst = (char *) HeapAlloc( GetProcessHeap(), 0, alloc_size );
-    for (i = 0; i < alloc_size; i++)
-        src[i] = 0xcafedead + i;
-
     SetLastError(0xdeadbeef);
     addr1 = VirtualAllocEx(hProcess, NULL, alloc_size, MEM_COMMIT,
                            PAGE_EXECUTE_READWRITE);
@@ -71,6 +66,12 @@ static void test_VirtualAllocEx(void)
         trace("VirtualAllocEx is not implemented, skipping the test\n");
         return;
     }
+
+    src = (char *) HeapAlloc( GetProcessHeap(), 0, alloc_size );
+    dst = (char *) HeapAlloc( GetProcessHeap(), 0, alloc_size );
+    for (i = 0; i < alloc_size; i++)
+        src[i] = 0xcafedead + i;
+
     todo_wine ok(addr1 != NULL, "VirtualAllocEx error %u\n", GetLastError());
     b = WriteProcessMemory(hProcess, addr1, src, alloc_size, &bytes_written);
     ok(b && (bytes_written == alloc_size), "%lu bytes written\n",




More information about the wine-cvs mailing list