[usrmarshal] Avoid a memory leak; spotted by Valgrind.

Jeremy White jwhite at codeweavers.com
Fri Dec 11 09:25:25 CST 2009


---
 dlls/oleaut32/tests/usrmarshal.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/dlls/oleaut32/tests/usrmarshal.c b/dlls/oleaut32/tests/usrmarshal.c
index d57802c..9756eb4 100644
--- a/dlls/oleaut32/tests/usrmarshal.c
+++ b/dlls/oleaut32/tests/usrmarshal.c
@@ -396,7 +396,10 @@ static void test_marshal_LPSAFEARRAY(void)
             hr = SafeArrayGetElement(lpsa2, indices, &gotvalue);
             ok(hr == S_OK, "Failed to get bstr element at hres 0x%x\n", hr);
             if (hr == S_OK)
+            {
                 ok(VarBstrCmp(values[i], gotvalue, 0, 0) == VARCMP_EQ, "String %d does not match\n", i);
+                SysFreeString(gotvalue);
+            }
         }
 
         SysFreeString(values[i]);



More information about the wine-patches mailing list