[PATCH 1/3] scrobj/tests: Don't set site to NULL before it has been freed (Valgrind).

Sven Baars sven.wine at gmail.com
Sat Oct 19 14:20:52 CDT 2019


Signed-off-by: Sven Baars <sven.wine at gmail.com>
---
 dlls/scrobj/tests/scrobj.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/scrobj/tests/scrobj.c b/dlls/scrobj/tests/scrobj.c
index 023aa30f92..861725c41a 100644
--- a/dlls/scrobj/tests/scrobj.c
+++ b/dlls/scrobj/tests/scrobj.c
@@ -634,7 +634,6 @@ static HRESULT WINAPI ClassFactory_CreateInstance(IClassFactory *iface, IUnknown
     ok(!outer, "outer = %p\n", outer);
     ok(IsEqualGUID(&IID_IUnknown, riid), "unexpected riid %s\n", wine_dbgstr_guid(riid));
     *ppv = &ActiveScript;
-    site = NULL;
     return S_OK;
 }
 
@@ -750,6 +749,7 @@ static void register_script_object(BOOL do_register, const WCHAR *file_name)
     HRESULT hres;
 
     parse_flags = SCRIPTTEXT_ISPERSISTENT | SCRIPTTEXT_ISVISIBLE;
+    site = NULL;
 
     SET_EXPECT(CreateInstance);
     SET_EXPECT(QI_IActiveScriptParse);
-- 
2.17.1




More information about the wine-devel mailing list