pbutils: Fix vp9 mime mapping when caps when no chroma-site.
authorVíctor Manuel Jáquez Leal <vjaquez@igalia.com>
Mon, 14 Nov 2022 11:44:05 +0000 (12:44 +0100)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 16 Nov 2022 01:29:50 +0000 (01:29 +0000)
It's possible to have 4:2:0 chroma format without chroma-site in caps, but it
isn't validated by the current code. This patch fixes it.

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

subprojects/gst-plugins-base/gst-libs/gst/pbutils/codec-utils.c

index 0167210..a89f3b0 100644 (file)
@@ -2520,7 +2520,10 @@ vp9_caps_get_mime_codec (GstCaps * caps)
     GstVideoChromaSite chroma_site;
 
     chroma_site_str = gst_structure_get_string (caps_st, "chroma-site");
-    chroma_site = gst_video_chroma_site_from_string (chroma_site_str);
+    if (chroma_site_str)
+      chroma_site = gst_video_chroma_site_from_string (chroma_site_str);
+    else
+      chroma_site = GST_VIDEO_CHROMA_SITE_UNKNOWN;
     if (chroma_site == GST_VIDEO_CHROMA_SITE_V_COSITED) {
       chroma_format = 0;
     } else if (chroma_site == GST_VIDEO_CHROMA_SITE_COSITED) {