From: Sven Baars Subject: [PATCH 1/3] scrobj/tests: Don't set site to NULL before it has been freed (Valgrind). Message-Id: <20191019192054.27156-1-sven.wine@gmail.com> Date: Sat, 19 Oct 2019 21:20:52 +0200 Signed-off-by: Sven Baars --- 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