On Sat, Feb 11, 2017 at 4:44 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > Bug-Id: 1017 > CC: libav-sta...@libav.org > --- > > libavformat/mov.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index 2810960..001abcc 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -1911,6 +1911,12 @@ static int mov_read_stsd(MOVContext *c, AVIOContext > *pb, MOVAtom atom) > avio_rb24(pb); /* flags */ > entries = avio_rb32(pb); > > + if (sc->extradata) { > + av_log(c->fc, AV_LOG_ERROR, > + "A stsd atom already found for the track.\n");
atom is the "mov" way of saying "box" and that might cause confusion, so I would rephrase the error message just "Duplicate stsd found in this track" Related, I'd drop "boxes" from the commit title as well. Otherwise OK. -- Vittorio _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel