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

Jacek Caban jacek at codeweavers.com
Mon Oct 21 05:17:29 CDT 2019


Hi Sven,


On 19/10/2019 15:20, Sven Baars wrote:
> @@ -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;


Do we need to set it here? It seems to me that Close() should already 
take care of that.


Thanks,

Jacek




More information about the wine-devel mailing list