Re: [PATCH 1/2] mfplat: Add NULL pointer checks to IMFMediaEvent get methods.

[Nikolay Sivov]

On 11/17/18 11:22 PM, Sven Baars wrote:
> @@ -2125,6 +2134,9 @@ static HRESULT WINAPI mfmediaevent_GetValue(IMFMediaEvent *iface, PROPVARIANT *v
>   {
>       mfmediaevent *This = impl_from_IMFMediaEvent(iface);
>
> +    if (!value)
> +        return E_INVALIDARG;
> +
>       PropVariantCopy(value, &This->value);
>
> return S_OK;

Hi, Sven.

Could you fix PropVariantCopy() to deal with NULL destination instead? 
According to my test it should return exactly E_INVALIDARG.