[FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

[Paul B Mahol]

On 3/2/15, Clement Boesch <u at pkh.me> wrote:
> On Mon, Mar 02, 2015 at 08:31:16AM +0000, Paul B Mahol wrote:
>> On 2/28/15, Clement Boesch <u at pkh.me> wrote:
>> > On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote:
>> >> ---
>> >>  libavcodec/avuienc.c            | 2 +-
>> >>  libavcodec/libopenjpegenc.c     | 4 ++--
>> >>  libavcodec/libtheoraenc.c       | 2 +-
>> >>  libavcodec/libutvideoenc.cpp    | 2 +-
>> >>  libavcodec/libvpxenc.c          | 2 +-
>> >>  libavcodec/proresenc_anatoliy.c | 2 +-
>> >>  libavcodec/proresenc_kostya.c   | 2 +-
>> >>  libavcodec/r210enc.c            | 2 +-
>> >>  libavcodec/utvideoenc.c         | 2 +-
>> >>  libavcodec/v210enc.c            | 2 +-
>> >>  libavcodec/v308enc.c            | 2 +-
>> >>  libavcodec/v408enc.c            | 2 +-
>> >>  libavcodec/v410enc.c            | 2 +-
>> >>  libavcodec/xfaceenc.c           | 2 +-
>> >>  libavcodec/y41penc.c            | 2 +-
>> >>  libavcodec/yuv4enc.c            | 2 +-
>> >>  16 files changed, 17 insertions(+), 17 deletions(-)
>> >>
>> >
>> > Mmh actually this actually fails for at least r210...
>> >
>> It does not fail here.
>> Patchset ok.
>
> I was confused by the output of FATE, it's not in r210, it's in prores:
>     *avctx->coded_frame           = *pic;

In that case the allocation is not needed.

>
> This definitely causes problems. I need to check other encoders. I don't
> know what to do to fix this particular case.
>
> [...]
>
> --
> Clement B.
>