Re: [PATCH 0/5] MR5971: winegstreamer/video_encoder: More of encoder implementation.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/video_encoder.c:
>  for (i = 0; i < encoder->input_type_count; ++i)
>      if (IsEqualGUID(&subtype, encoder->input_types[i]))
>          break;
>  if (i == encoder->input_type_count)
>      return MF_E_INVALIDMEDIATYPE;
>
> - if (FAILED(IMFMediaType_GetUINT64(type, &MF_MT_FRAME_SIZE, &ratio))
> -
>        || FAILED(IMFMediaType_GetUINT64(type, &MF_MT_FRAME_RATE, &ratio)))
> - if (FAILED(hr = create_input_type(encoder, &subtype, &good_input_type)))
> -
>    return hr;
> - if (FAILED(hr = IMFMediaType_Compare(good_input_type, (IMFAttributes *)type,
> -
>        MF_ATTRIBUTES_MATCH_OUR_ITEMS, &result)))
> -
>    return hr;
> - if (!result)
>     return MF_E_INVALIDMEDIATYPE;

You're leaking good_input_type here.