sdp: fix wrong error message for missing clock-rate in caps
authorStephan Seitz <sseitz@nvidia.com>
Sat, 26 Aug 2023 13:12:05 +0000 (15:12 +0200)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 26 Aug 2023 18:35:32 +0000 (19:35 +0100)
When using `gst_sdp_media_set_media_from_caps` on `application/x-rtp` caps
without `clock-rate` it wrongly reports missing payload type even if `payload`
is present in the caps.

This seems to be a copy&paste error from the error message for missing payload
type.

When using payload=10, both `clock-rate` and some other media properties are
defined by the RTP standard so I was wondering whether I could omit `clock-rate`
and was confused about the error message.

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

subprojects/gst-plugins-base/gst-libs/gst/sdp/gstsdpmessage.c

index c4dde75..2b40bd9 100644 (file)
@@ -3824,7 +3824,7 @@ gst_sdp_media_set_media_from_caps (const GstCaps * caps, GstSDPMedia * media)
 
   /* get clock-rate, media type and params for the rtpmap attribute */
   if (!gst_structure_get_int (s, "clock-rate", &caps_rate)) {
-    GST_ERROR ("ignoring stream without payload type");
+    GST_ERROR ("ignoring stream without clock rate");
     goto error;
   }
   caps_enc = gst_structure_get_string (s, "encoding-name");