On Wed, Nov 08, 2017 at 07:22:54PM -0500, Sean McGovern wrote:
> --- a/libavcodec/smacker.c
> +++ b/libavcodec/smacker.c
> @@ -133,8 +134,13 @@ static int smacker_decode_tree(BitstreamContext *bc,
> HuffContext *hc,
> static int smacker_decode_bigtree(BitstreamContext *bc, HuffContext *hc,
> - DBCtx *ctx)
> + DBCtx *ctx, int length)
> {
> + if (length > SMKTREE_DECODE_BIG_MAX_RECURSION) { // Larger length can
> cause segmentation faults due to too deep recursion.
> + av_log(NULL, AV_LOG_ERROR, "length too long\n");
"Maximum bigtree recursion level exceeded.\n"
I'd also move the comment above the if-statement.
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
