Re: [PATCH 1/3] mshtml: Add a null parameter check in do_query_service().

[Jacek Caban]

Hi Zebediah,

On 14.09.2017 20:42, Zebediah Figura wrote:
> diff --git a/dlls/mshtml/main.c b/dlls/mshtml/main.c
> index af716d1680..43270b9cc8 100644
> --- a/dlls/mshtml/main.c
> +++ b/dlls/mshtml/main.c
> @@ -190,6 +190,9 @@ HRESULT do_query_service(IUnknown *unk, REFGUID guid_service, REFIID riid, void
>      IServiceProvider *sp;
>      HRESULT hres;
>
> +    if (!unk)
> +        return E_FAIL;

I think it would be cleaner to add NULL check to caller.


Thanks,

Jacek