[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