From df06a38c5bc7fdcb741b387b6bdfd3377d3f3164 Mon Sep 17 00:00:00 2001 From: Martin Storsjo Date: Mon, 25 May 2015 14:42:53 +0300 Subject: [PATCH] Include space for chroma in the padding allocated in ih264e_get_total_pic_buf_size Chroma is already accounted for in e.g. num_samples by multiplying the number of luma samples by 3/2, so include it in the padding calculation as well. This fixes encoding 176x144 at level 1.0 - normally this isn't an issue since the padding is a bit overestimated, but for this particular resolution/level combination, it fails unless this is calculated properly. Change-Id: Id5df6e5aee4d3d78f4c28a3b7d5fed6fe42a1f2c --- encoder/ih264e_utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/encoder/ih264e_utils.c b/encoder/ih264e_utils.c index f0086cb..44e4cc0 100644 --- a/encoder/ih264e_utils.c +++ b/encoder/ih264e_utils.c @@ -366,7 +366,7 @@ WORD32 ih264e_get_total_pic_buf_size(WORD32 pic_size, size = num_samples * max_num_bufs; /* Account for padding area */ - size += ((pad * pad) + pad * (max_wd + min_ht)) * max_num_bufs; + size += ((pad * pad) + pad * (max_wd + min_ht)) * 3 / 2 * max_num_bufs; return size; }