Hans Leidekker : userenv/tests: Fix some memory leaks.

Alexandre Julliard julliard at winehq.org
Mon Dec 7 10:26:17 CST 2009


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

Author: Hans Leidekker <hans at codeweavers.com>
Date:   Mon Dec  7 09:19:23 2009 +0100

userenv/tests: Fix some memory leaks.

Found by Valgrind.

---

 dlls/userenv/tests/userenv.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/dlls/userenv/tests/userenv.c b/dlls/userenv/tests/userenv.c
index 7c1228b..9003354 100644
--- a/dlls/userenv/tests/userenv.c
+++ b/dlls/userenv/tests/userenv.c
@@ -145,6 +145,7 @@ static void test_create_env(void)
                 todo_wine expect_env(TRUE, r, common_vars[i].name);
             else
                 expect_env(TRUE, r, common_vars[i].name);
+            if (r) HeapFree(GetProcessHeap(), 0, st);
         }
     }
 
@@ -164,6 +165,7 @@ static void test_create_env(void)
                     todo_wine expect_env(TRUE, r, common_post_nt4_vars[i].name);
                 else
                     expect_env(TRUE, r, common_post_nt4_vars[i].name);
+                if (r) HeapFree(GetProcessHeap(), 0, st);
             }
         }
     }
@@ -178,17 +180,23 @@ static void test_create_env(void)
                 todo_wine expect_env(TRUE, r, htok_vars[i].name);
             else
                 expect_env(TRUE, r, htok_vars[i].name);
+            if (r) HeapFree(GetProcessHeap(), 0, st);
         }
     }
 
     r = get_env(env[0], "WINE_XYZZY", &st);
     expect(FALSE, r);
+
     r = get_env(env[1], "WINE_XYZZY", &st);
     expect(FALSE, r);
+
     r = get_env(env[2], "WINE_XYZZY", &st);
     expect(TRUE, r);
+    if (r) HeapFree(GetProcessHeap(), 0, st);
+
     r = get_env(env[3], "WINE_XYZZY", &st);
     expect(TRUE, r);
+    if (r) HeapFree(GetProcessHeap(), 0, st);
 }
 
 START_TEST(userenv)




More information about the wine-cvs mailing list