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

Michael Niedermayer michael at niedermayer.cc
Mon Feb 12 00:59:20 EET 2018


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.
That is this looks like a unrelated issue of the Parser in general not printing 
a proper name

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180211/14e8e00e/attachment.sig>


More information about the ffmpeg-devel mailing list