[FFmpeg-devel] [PATCH 1/2] avcodec/mpeg4videodec: Ignore multiple VOL headers

Carl Eugen Hoyos ceffmpeg at gmail.com
Mon Feb 12 16:03:38 EET 2018


2018-02-11 23:59 GMT+01:00 Michael Niedermayer <michael at niedermayer.cc>:
> On Sun, Feb 11, 2018 at 08:01:36PM +0100, Carl Eugen Hoyos wrote:
>> 2018-02-09 22:24 GMT+01:00 Michael Niedermayer <michael at niedermayer.cc>:
>> > Fixes: Ticket7005
>> >
>> > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
>> > ---
>> >  libavcodec/mpeg4videodec.c | 4 ++--
>> >  1 file changed, 2 insertions(+), 2 deletions(-)
>> >
>> > diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c
>> > index 756753e2fc..19210d97fe 100644
>> > --- a/libavcodec/mpeg4videodec.c
>> > +++ b/libavcodec/mpeg4videodec.c
>> > @@ -2707,8 +2707,8 @@ int ff_mpeg4_decode_picture_header(Mpeg4DecContext *ctx, GetBitContext *gb)
>> >
>> >          if (startcode >= 0x120 && startcode <= 0x12F) {
>> >              if (vol) {
>> > -                av_log(s->avctx, AV_LOG_ERROR, "Multiple VOL headers");
>> > -                return AVERROR_INVALIDDATA;
>> > +                av_log(s->avctx, AV_LOG_WARNING, "Ignoring multiple VOL headers\n");
>> > +                continue;
>> >              }
>>
>> Is it expected that the warning is printed as following on decoding
>> now (context switches between NULL and mpeg4)?
>
> It doesnt really switch, i guess the NULL is from the AVParser which does not
> have a codec setup and the context_to_name() uses the codec.

Thank you, Carl Eugen


More information about the ffmpeg-devel mailing list