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

Nikolay Sivov nsivov at codeweavers.com
Sat Nov 17 14:33:16 CST 2018


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.




More information about the wine-devel mailing list