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

[Sven Baars]

On 21-10-19 12:17, Jacek Caban wrote:
> 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
> 
Hi Jacek,

I did that to initialize it to NULL before it gets used in the other
methods. Else it would be uninitialized.

Best,
Sven