[FFmpeg-devel] [PATCH] avutil/imgutils: don't fill data pointers for missing planes

James Almer jamrial at gmail.com
Fri Jul 24 21:08:13 EEST 2020


The size for a previous plane doesn't signal the presence of another after it.
If the plane is present, av_image_fill_plane_sizes() will have returned a size
for it.

Fixes a regression since 3a8e927176.

Reported-by: Imad R. Faiad <irfaiad at gmail.com>
Signed-off-by: James Almer <jamrial at gmail.com>
---
 libavutil/imgutils.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavutil/imgutils.c b/libavutil/imgutils.c
index 4096c0da66..e592f7976a 100644
--- a/libavutil/imgutils.c
+++ b/libavutil/imgutils.c
@@ -167,7 +167,7 @@ int av_image_fill_pointers(uint8_t *data[4], enum AVPixelFormat pix_fmt, int hei
     memset(data , 0, sizeof(data[0])*4);
 
     data[0] = ptr;
-    for (i = 1; i < 4 && sizes[i - 1] > 0; i++)
+    for (i = 1; i < 4 && sizes[i]; i++)
         data[i] = data[i - 1] + sizes[i - 1];
 
     return ret;
-- 
2.27.0



More information about the ffmpeg-devel mailing list