[FFmpeg-devel] [PATCH 4/8] ffmdec: don't set extradata_size without allocating extradata

Michael Niedermayer michaelni at gmx.at
Mon Mar 9 03:43:53 CET 2015


On Mon, Mar 09, 2015 at 12:03:54AM +0100, Andreas Cadhalpun wrote:
> Hi,
> 
> attached patch fixes crashes with ffm files containing an
> extradata_size setting, but without extradata.
> 
> Such files are created by the ffm muxer, e.g. with for the asv1 codec.
> 
> Best regards,
> Andreas

>  ffmdec.c |   14 +++++++++++++-
>  1 file changed, 13 insertions(+), 1 deletion(-)
> beff822dbd6683a15edfc42702af8955475b0172  0004-ffmdec-don-t-set-extradata_size-without-allocating-e.patch
> From b3da247975db1bc0dd86805f2830aa0b718fd355 Mon Sep 17 00:00:00 2001
> From: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> Date: Sun, 8 Mar 2015 23:24:40 +0100
> Subject: [PATCH 4/8] ffmdec: don't set extradata_size without allocating
>  extradata
> 
> When extradata_size is not 0, but extradata is NULL, code trying to use
> extradata crashes.
> 
> If there is any extradata, extradata_size is set in ff_get_extradata.

ive posted a patch that removes the extradata_size field from the
AVOption table, i think thats better if it works

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150309/352ecd2e/attachment.asc>


More information about the ffmpeg-devel mailing list