Re: [PATCH v2 0/1] MR3608: mshtml: check for err before trying to hold plugin host ref

[Gabriel Ivăncescu (＠insn)]

Gabriel Ivăncescu (@insn) commented about dlls/mshtml/npplugin.c:
>  }
>
>  instance->pdata = container->plugin_host;
> - IOleClientSite_AddRef(&container->plugin_host->IOleClientSite_iface);
> - if(!err)
> -
>    IOleClientSite_AddRef(&container->plugin_host->IOleClientSite_iface);

I'd personally check for `container->plugin_host` being non-NULL here instead of checking for err, but it doesn't really matter, depends how Jacek feels about it.

Minor nitpick: Please capitalize first word in the commit title (`check`→`Check`) and add a period at the end.