Re: [PATCH 1/5] gdiplus: Refactor to split up encode_image_wic().

[Vincent Povirk (they/them)]

> +GpStatus terminate_encoder_wic(IWICBitmapEncoder *encoder)
> +{
> +    HRESULT hr = IWICBitmapEncoder_Commit(encoder);
> +    IWICBitmapEncoder_Release(encoder);
> +    return hresult_to_status(hr);
> +}

I don't like having the Release call separated from clearing or
freeing the stored reference to the interface. I think we're likely to
miss a use-after-free this way.

I'd suggest either making this function take a GpImage* so we can
clear the reference, or moving the Release call out of it.