Re: [PATCH 0/5] MR6721: mshtml: Fix deletion of global window props.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/script.c:
> -
>        return E_UNEXPECTED;
> -
> -
>    if(!IsEqualGUID(&CLSID_JScript, &prop->script_host->guid) ||
> -
>       IActiveScript_QueryInterface(prop->script_host->script, &IID_IWineJScript, (void **)&jscript) != S_OK)
> -
>        return S_OK;
> -
> -
>    hres = IWineJScript_GlobalPropExists(jscript, prop->id);
> -
>    IWineJScript_Release(jscript);
> -
>    return hres;
> - }
> - case GLOBAL_ELEMENTVAR: {
> -
>    IHTMLElement *elem;
> -
>    BSTR bstr;
> -
> -
>    if(!(bstr = SysAllocString(prop->name)))
> -
>        return E_OUTOFMEMORY;

I think it would be better to just store the name as `BSTR` instead.