rtpopusdepay: assume 2 channels if sprop-stereo is missing
authorTristan Matthews <tmatth@videolan.org>
Wed, 6 Apr 2022 13:46:30 +0000 (09:46 -0400)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Fri, 8 Apr 2022 13:11:25 +0000 (13:11 +0000)
Fixes #1064

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

subprojects/gst-plugins-good/gst/rtp/gstrtpopusdepay.c

index 26434e8..3dc4fba 100644 (file)
@@ -206,8 +206,10 @@ gst_rtp_opus_depay_setcaps (GstRTPBaseDepayload * depayload, GstCaps * caps)
         GST_WARNING_OBJECT (depayload, "Unknown sprop-stereo value '%s'",
             sprop_stereo);
     } else {
         GST_WARNING_OBJECT (depayload, "Unknown sprop-stereo value '%s'",
             sprop_stereo);
     } else {
-      /* sprop-stereo defaults to mono as per RFC 7587. */
-      gst_caps_set_simple (srccaps, "channels", G_TYPE_INT, 1, NULL);
+      /* Although sprop-stereo defaults to mono as per RFC 7587, this just means
+         that the signal is likely mono and can be safely downmixed, it may
+         still be stereo at times. */
+      gst_caps_set_simple (srccaps, "channels", G_TYPE_INT, 2, NULL);
     }
   }
 
     }
   }