[FFmpeg-devel] [PATCH] mxfdec: do not try to use an obviously broken index.
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Sat Dec 10 13:18:50 CET 2011
On Sat, Dec 10, 2011 at 12:51:42PM +0100, Tomas Härdin wrote:
> On Sat, 2011-12-10 at 11:52 +0100, Reimar Döffinger wrote:
> > Also fixes a memleak.
> >
> > Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> > ---
> > libavformat/mxfdec.c | 27 +++++++++++++++++++++++----
> > 1 files changed, 23 insertions(+), 4 deletions(-)
> >
> > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> > index 7b32a5a..d914920 100644
> > --- a/libavformat/mxfdec.c
> > +++ b/libavformat/mxfdec.c
> > @@ -200,6 +200,7 @@ typedef struct {
> > int current_edit_unit;
> > int current_stream;
> > int d10;
> > + int broken_index;
>
> Maybe just merge these two into a common variable called
> "demux_indexless" or something? Not a bit deal though.
I do not know what that d10 thing does, but it is also used
in one more place so I doubt that merging it will work as expected.
Personally I suspect it is a lazy hack because nobody was willing
to find and fix the real issue and "ok, let's just make a special
case for this file" seemed like a good idea.
More information about the ffmpeg-devel
mailing list