gsth264parse: Fix create() width and height calculation

Ignore the frame_cropping flag set in the SPS and instead
assign the maximum width and height.

Also, always set SPS width and height instead of override
by upstream components

Signed-off-by: Ramprasad <x0038811@ti.com>
Signed-off-by: Karthik Ramanan <a0393906@ti.com>
Signed-off-by: Pooja Prajod <a0132412@ti.com>

1 files changed, 6 insertions, 14 deletions

diff --git a/gst/videoparsers/gsth264parse.c b/gst/videoparsers/gsth264parse.c
index 1aeecf1..3d49f4a 100644
--- a/gst/videoparsers/gsth264parse.c
+++ b/gst/videoparsers/gsth264parse.c
@@ -1712,13 +1712,8 @@ gst_h264_parse_update_src_caps (GstH264Parse * h264parse, GstCaps * caps)
     gint fps_num, fps_den;
     gint par_n, par_d;
 
-    if (sps->frame_cropping_flag) {
-      crop_width = sps->crop_rect_width;
-      crop_height = sps->crop_rect_height;
-    } else {
-      crop_width = sps->width;
-      crop_height = sps->height;
-    }
+    crop_width = (sps->crop_rect_width > sps->width)? sps->crop_rect_width: sps->width;
+    crop_height = (sps->crop_rect_height > sps->height)? sps->crop_rect_height: sps->height;
 
     if (G_UNLIKELY (h264parse->width != crop_width ||
             h264parse->height != crop_height)) {
@@ -1752,7 +1747,7 @@ gst_h264_parse_update_src_caps (GstH264Parse * h264parse, GstCaps * caps)
     }
 
     if (G_UNLIKELY (modified || h264parse->update_caps)) {
-      gint width, height;
+      gint width=0, height=0;
       GstClockTime latency;
 
       const gchar *caps_mview_mode = NULL;
@@ -1764,16 +1759,13 @@ gst_h264_parse_update_src_caps (GstH264Parse * h264parse, GstCaps * caps)
 
       caps = gst_caps_copy (sink_caps);
 
-      /* sps should give this but upstream overrides */
       if (s && gst_structure_has_field (s, "width"))
         gst_structure_get_int (s, "width", &width);
-      else
-        width = h264parse->width;
-
+      width = (width > h264parse->width)? width: h264parse->width;
+     
       if (s && gst_structure_has_field (s, "height"))
         gst_structure_get_int (s, "height", &height);
-      else
-        height = h264parse->height;
+      height = (height > h264parse->height)? height: h264parse->height;
 
       if (s == NULL ||
           !gst_structure_get_fraction (s, "pixel-aspect-ratio", &par_n,