userenv/tests: Fix some memory leaks.

Hans Leidekker hans at codeweavers.com
Mon Dec 7 02:19:23 CST 2009


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)
-- 
1.6.3.3




More information about the wine-patches mailing list