Re: [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