Vincent Povirk <madewokh...@gmail.com> wrote:

>  static HRESULT WINAPI BitmapScaler_GetSize(IWICBitmapScaler *iface,
>      UINT *puiWidth, UINT *puiHeight)
>  {
> -    FIXME("(%p,%p,%p): stub\n", iface, puiWidth, puiHeight);
> +    BitmapScaler *This = impl_from_IWICBitmapScaler(iface);
> +    TRACE("(%p,%p,%p)\n", iface, puiWidth, puiHeight);
>  
> -    return E_NOTIMPL;
> +    if (!This->source)
> +        return WINCODEC_ERR_WRONGSTATE;
> +
> +    *puiWidth = This->width;
> +    *puiHeight = This->height;
> +
> +    return S_OK;

This patch and remaning patches in the sequence completely miss any
parameter checking.

-- 
Dmitry.


Reply via email to