Sven Baars : taskschd/tests: Fix some memory leaks (Valgrind).

Alexandre Julliard julliard at winehq.org
Tue Jan 15 16:04:39 CST 2019


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

Author: Sven Baars <sven.wine at gmail.com>
Date:   Tue Jan 15 13:56:51 2019 +0100

taskschd/tests: Fix some memory leaks (Valgrind).

Signed-off-by: Sven Baars <sven.wine at gmail.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/taskschd/tests/scheduler.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/dlls/taskschd/tests/scheduler.c b/dlls/taskschd/tests/scheduler.c
index 5fc3aab..639555e 100644
--- a/dlls/taskschd/tests/scheduler.c
+++ b/dlls/taskschd/tests/scheduler.c
@@ -544,6 +544,7 @@ static void test_FolderCollection(void)
             ok(!lstrcmpW(bstr, Folder1), "expected Folder1, got %s\n", wine_dbgstr_w(bstr));
         else
             ok(!lstrcmpW(bstr, Folder2), "expected Folder2, got %s\n", wine_dbgstr_w(bstr));
+        SysFreeString(bstr);
 
         ITaskFolder_Release(subfolder);
     }
@@ -571,6 +572,7 @@ static void test_FolderCollection(void)
 
     hr = IUnknown_QueryInterface(unknown, &IID_IUnknown, (void **)&enumvar);
     ok(hr == S_OK, "QueryInterface error %#x\n", hr);
+    IUnknown_Release(unknown);
 
     hr = IEnumVARIANT_Skip(enumvar, 0);
     ok(hr == S_OK, "expected S_OK, got %#x\n", hr);
@@ -605,6 +607,7 @@ static void test_FolderCollection(void)
         ok(!lstrcmpW(bstr, Folder1), "expected Folder1, got %s\n", wine_dbgstr_w(bstr));
     else
         ok(!lstrcmpW(bstr, Folder2), "expected Folder2, got %s\n", wine_dbgstr_w(bstr));
+    SysFreeString(bstr);
     IDispatch_Release(V_DISPATCH(&var[0]));
 
     memset(var, 0, sizeof(var));
@@ -650,6 +653,7 @@ static void test_FolderCollection(void)
             ok(!lstrcmpW(bstr, Folder1), "expected Folder1, got %s\n", wine_dbgstr_w(bstr));
         else
             ok(!lstrcmpW(bstr, Folder2), "expected Folder2, got %s\n", wine_dbgstr_w(bstr));
+        SysFreeString(bstr);
 
         ITaskFolder_Release(subfolder);
     }
@@ -1049,7 +1053,10 @@ static void test_settings_v1(ITaskDefinition *taskdef, struct settings *test, st
     if (!test->execution_time_limit[0])
         ok(bstr == NULL, "expected NULL, got %s\n", wine_dbgstr_w(bstr));
     else
+    {
         ok(!lstrcmpW(bstr, test->execution_time_limit), "expected %s, got %s\n", wine_dbgstr_w(test->execution_time_limit), wine_dbgstr_w(bstr));
+        SysFreeString(bstr);
+    }
 
     hr = ITaskSettings_get_Enabled(set, &vbool);
     ok(hr == S_OK, "expected S_OK, got %#x\n", hr);




More information about the wine-cvs mailing list