Nikolay Sivov : opcservices/tests: Fix some leaks in tests (Valgrind).

Alexandre Julliard julliard at winehq.org
Thu Sep 13 15:14:47 CDT 2018


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

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Thu Sep 13 09:21:18 2018 +0300

opcservices/tests: Fix some leaks in tests (Valgrind).

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/opcservices/tests/opcservices.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/dlls/opcservices/tests/opcservices.c b/dlls/opcservices/tests/opcservices.c
index a9c25a5..0258bf2 100644
--- a/dlls/opcservices/tests/opcservices.c
+++ b/dlls/opcservices/tests/opcservices.c
@@ -75,6 +75,7 @@ static void test_package(void)
     hr = IOpcPackage_GetPartSet(package, &partset2);
     ok(SUCCEEDED(hr), "Failed to create a part set, hr %#x.\n", hr);
     ok(partset == partset2, "Expected same part set instance.\n");
+    IOpcPartSet_Release(partset2);
 
     /* CreatePart */
     hr = IOpcFactory_CreatePartUri(factory, uriW, &part_uri);
@@ -161,6 +162,7 @@ todo_wine {
     IOpcRelationshipSet_Release(relset);
     IOpcRelationshipSet_Release(relset2);
 
+    IOpcPartSet_Release(partset);
     IOpcPackage_Release(package);
 
     /* Root uri */
@@ -322,6 +324,8 @@ static void test_relationship(void)
     hr = IOpcRelationshipSet_CreateRelationship(rels, NULL, typeW, target_uri2, OPC_URI_TARGET_MODE_INTERNAL, &rel);
 todo_wine
     ok(hr == OPC_E_INVALID_RELATIONSHIP_TARGET, "Unexpected hr %#x.\n", hr);
+    if (hr == S_OK)
+        IOpcRelationship_Release(rel);
 
     hr = IOpcRelationshipSet_CreateRelationship(rels, NULL, typeW, target_uri, OPC_URI_TARGET_MODE_INTERNAL, &rel);
     ok(SUCCEEDED(hr), "Failed to create relationship, hr %#x.\n", hr);




More information about the wine-cvs mailing list