[FFmpeg-devel] [PATCH] mxfdec: do not try to use an obviously broken index.

Tomas Härdin tomas.hardin at codemill.se
Sat Dec 10 12:51:42 CET 2011


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.

The rest of the patch looks OK. I was just about to post a patch that
only does the sorting once, in mxf_parse_structural_metadata(). I can
rebase it on top of this one though.

/Tomas



More information about the ffmpeg-devel mailing list