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.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3608#note_42721
Signed-off-by: Torge Matthies <openglfreak(a)googlemail.com>
--
v3: loader: Add Default, Failed, and LastKnownGood values to HKLM\System\Select.
server: Create link from HKLM\System\CurrentControlSet to ControlSet001.
advapi32/tests: Add test for CurrentControlSet link.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3563