Revert "video-frame: avoid possible out of bound memory access"

This reverts commit c4255f08f787ff6a41504b538fa4c2e911d40ec7.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2239>

diff --git a/subprojects/gst-plugins-base/gst-libs/gst/video/video-frame.c b/subprojects/gst-plugins-base/gst-libs/gst/video/video-frame.c
index a1bfe39..d920912 100644 (file)
--- a/subprojects/gst-plugins-base/gst-libs/gst/video/video-frame.c
+++ b/subprojects/gst-plugins-base/gst-libs/gst/video/video-frame.c
@@ -283,7 +283,7 @@ scale_tile_shifts (const GstVideoFormatInfo * finfo, gint plane, guint * ws,
     guint * hs)
 {
   gint comp[GST_VIDEO_MAX_COMPONENTS];
-  gint i = 1;
+  gint i;
 
   gst_video_format_info_component (finfo, plane, comp);
 
@@ -294,10 +294,8 @@ scale_tile_shifts (const GstVideoFormatInfo * finfo, gint plane, guint * ws,
   /* for each additional component in the same plane, double the tile width,
    * this should provide the appropriate tile size when the tile size varies
    * base on the subsampling. */
-  while (i < GST_VIDEO_MAX_COMPONENTS && comp[i] >= 0) {
+  for (i = 1; comp[i] >= 0; i++)
     *ws += 1;
-    i++;
-  }
 }