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

[James Almer]

ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Fri Jul 24 15:04:05 2020 -0300| [5ce47d0aad31eff27a8626dd99774623f4d1b7a0] | committer: James Almer

avutil/imgutils: don't fill data pointers for missing planes

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>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=5ce47d0aad31eff27a8626dd99774623f4d1b7a0
---

 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;