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

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Sun Nov 20 01:49:34 EET 2016


On 20.11.2016 00:25, Luca Barbato wrote:
> On 19/11/2016 17:27, Andreas Cadhalpun wrote:
>> This fixes segmentation faults due to stack-overflow caused by too
>> deep recursion.
> 
> You shouldn't be able to use hc->current for the same purpose?

That's what I tried first, but it doesn't work, since hc->current
can be much larger than the actual recursion depth.

> Another option is to write that function as a nested loop I guess.

Possibly.

Best regards,
Andreas



More information about the ffmpeg-devel mailing list