Dan Hipschman : qmgr/tests: Fix minor potential memory leaks.
Alexandre Julliard
julliard at winehq.org
Wed Apr 16 08:09:50 CDT 2008
Module: wine
Branch: master
Commit: cacd1c448e633852c77bfa87572aa12895d7e86a
URL: http://source.winehq.org/git/wine.git/?a=commit;h=cacd1c448e633852c77bfa87572aa12895d7e86a
Author: Dan Hipschman <dsh at linux.ucla.edu>
Date: Tue Apr 15 11:08:48 2008 -0700
qmgr/tests: Fix minor potential memory leaks.
---
dlls/qmgr/tests/enum_jobs.c | 3 +++
dlls/qmgr/tests/job.c | 6 ++++++
2 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/dlls/qmgr/tests/enum_jobs.c b/dlls/qmgr/tests/enum_jobs.c
index 2b73631..f95404c 100644
--- a/dlls/qmgr/tests/enum_jobs.c
+++ b/dlls/qmgr/tests/enum_jobs.c
@@ -209,6 +209,7 @@ static void test_Next_walkList_2(void)
if(hres != S_OK)
{
skip("Unable to get file from test_enumJobs\n");
+ HeapFree(GetProcessHeap(), 0, jobs);
return;
}
ok(fetched == test_jobCountB, "Next returned the incorrect number of jobs: %08x\n", hres);
@@ -219,6 +220,8 @@ static void test_Next_walkList_2(void)
if (jobs[i])
IBackgroundCopyFile_Release(jobs[i]);
}
+
+ HeapFree(GetProcessHeap(), 0, jobs);
}
/* Test Next Error conditions */
diff --git a/dlls/qmgr/tests/job.c b/dlls/qmgr/tests/job.c
index 52d4e82..36201f8 100644
--- a/dlls/qmgr/tests/job.c
+++ b/dlls/qmgr/tests/job.c
@@ -416,6 +416,8 @@ static void test_CompleteLocalURL(void)
if (!urlA || !urlB)
{
skip("Unable to allocate memory for URLs\n");
+ HeapFree(GetProcessHeap(), 0, urlA);
+ HeapFree(GetProcessHeap(), 0, urlB);
return;
}
@@ -428,6 +430,8 @@ static void test_CompleteLocalURL(void)
if (hres != S_OK)
{
skip("Unable to add file to job\n");
+ HeapFree(GetProcessHeap(), 0, urlA);
+ HeapFree(GetProcessHeap(), 0, urlB);
return;
}
@@ -435,6 +439,8 @@ static void test_CompleteLocalURL(void)
if (hres != S_OK)
{
skip("Unable to add file to job\n");
+ HeapFree(GetProcessHeap(), 0, urlA);
+ HeapFree(GetProcessHeap(), 0, urlB);
return;
}
More information about the wine-cvs
mailing list