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

Clément Bœsch ubitux at gmail.com
Tue Dec 20 13:49:57 CET 2011


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.

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111220/951329b5/attachment.asc>


More information about the ffmpeg-devel mailing list