[FFmpeg-devel] [PATCH 1/2] mxfdec: Make sure mxf->nb_index_tables > 0 in mxf_packet_timestamps()

[Dave Rice]

On Dec 20, 2011, at 7:49 AM, Clément Bœsch wrote:

> On Tue, Dec 20, 2011 at 12:03:24PM +0100, Tomas Härdin wrote:
>> I was alerted to another issue on IRC, which this patch fixes. The
>> second patch is one I've had laying around.
>> Both patches also pushed to
>> https://github.com/Tjoppen/FFmpeg/tree/mxf_fixes_20111220
>> 
>> /Tomas
> 
>> From 1888bd1b6d2bedc7bbded91d3a8d236236b5aa3c Mon Sep 17 00:00:00 2001
>> From: =?UTF-8?q?Tomas=20H=C3=A4rdin?= <tomas.hardin at codemill.se>
>> Date: Tue, 20 Dec 2011 11:22:23 +0100
>> Subject: [PATCH 1/2] mxfdec: Make sure mxf->nb_index_tables > 0 in
>> mxf_packet_timestamps()
>> 
>> Only the OPAtom demuxing logic is guaranteed to have index tables, meaning OP1a
>> files that lack an index would cause SIGSEGV.
>> ---
>> libavformat/mxfdec.c |    5 +++++
>> 1 files changed, 5 insertions(+), 0 deletions(-)
>> 
> 
> I confirm this fix the crash I expect with a sample here, thanks.

This patch also fixed the issue that I mentioned on IRC yesterday, thanks much.
Dave Rice

> -- 
> Clément B.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel