[advapi32/tests] Free memory after use

Paul Vriens paul.vriens.wine at gmail.com
Mon Dec 4 14:25:13 CST 2006


Hi,

I see some strange failures on our tests-site for advpack:

http://test.winehq.org/data/200612021000/2003_PV-W2K3-SE-IE7/advpack:advpack.txt

Most of them don't happen when I run the advpack tests only. This lead me to
the last one before that, which was security of advpack. Don't know if this solves
things but freeing after use is a good thing to do, even in tests.

Changelog
  Free memory after use

Cheers,

Paul.
---
 dlls/advapi32/tests/security.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c
index b651364..7711f97 100644
--- a/dlls/advapi32/tests/security.c
+++ b/dlls/advapi32/tests/security.c
@@ -1309,6 +1309,9 @@ static void test_LookupAccountName(void)
         ok(sid_size == 0, "Expected 0, got %d\n", sid_size);
         ok(domain_size == 0, "Expected 0, got %d\n", domain_size);
     }
+
+    HeapFree(GetProcessHeap(), 0, psid);
+    HeapFree(GetProcessHeap(), 0, domain);
 }
 
 START_TEST(security)
-- 
1.4.4.1




More information about the wine-patches mailing list