[1/5] msxml3/tests: Valgrind fix.

Michael Karcher wine at mkarcher.dialup.fu-berlin.de
Mon Oct 6 17:48:05 CDT 2008


sName is an uninitialized pointer if GetMemberName faild.
---
 dlls/msxml3/tests/domdoc.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c
index 1ad168b..0703f3d 100644
--- a/dlls/msxml3/tests/domdoc.c
+++ b/dlls/msxml3/tests/domdoc.c
@@ -1565,7 +1565,8 @@ static void test_getElementsByTagName(void)
 
         r = IDispatchEx_GetMemberName(dispex, dispid, &sName);
         ok(r == E_NOTIMPL, "expected E_NOTIMPL got %08x\n", r);
-        SysFreeString(sName);
+        if( !FAILED(r) )
+            SysFreeString(sName);
 
         r = IDispatchEx_GetNextDispID(dispex, fdexEnumDefault, DISPID_XMLDOM_NODELIST_RESET, &dispid);
         ok(r == E_NOTIMPL, "expected E_NOTIMPL got %08x\n", r);
-- 
1.5.6.5




More information about the wine-patches mailing list