[FFmpeg-devel] [PATCH] smacker: limit recursion depth of smacker_decode_bigtree

[Michael Niedermayer]

On Sat, Nov 19, 2016 at 02:29:35PM +0100, Andreas Cadhalpun wrote:
> This fixes segmentation faults due to stack-overflow caused by too deep
> recursion.
> 
> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> ---
>  libavcodec/smacker.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

breaks fate

[smackvid @ 0x3586b80] size 31232 too large
Input #0, smk, from 'fate/fate-suite//smacker/wetlogo.smk':
  Duration: 00:00:07.10, start: 0.000000, bitrate: 815 kb/s
    Stream #0:0: Video: smackvideo (SMK2 / 0x324B4D53), pal8, 320x200, 14.08 tbr, 14.08 tbn, 14.08 tbc
    Stream #0:1: Audio: smackaudio (SMKA / 0x414B4D53), 22050 Hz, mono, u8
[smackvid @ 0x358b860] size 31232 too large
Stream mapping:
  Stream #0:0 -> #0:0 (smackvideo (smackvid) -> rawvideo (native))
Error while opening decoder for input stream #0:0 : Invalid data found when processing input
make: *** [fate-smacker-video] Error 1

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161119/c530551f/attachment.sig>