rtph263depay: Fix framesize parsing
authorEdward Hervey <bilboed@bilboed.com>
Wed, 8 Apr 2015 09:17:31 +0000 (11:17 +0200)
committerEdward Hervey <bilboed@bilboed.com>
Wed, 8 Apr 2015 09:17:31 +0000 (11:17 +0200)
The string passed to the parsing function only contains a framesize, and
not <pt> + <framesize>

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=726416

gst/rtp/gstrtph263depay.c

index 749da53..a63b0b1 100644 (file)
@@ -141,24 +141,10 @@ gst_rtp_h263_parse_framesize (GstRTPBaseDepayload * filter,
     const gchar * media_attr, GstCaps * srccaps)
 {
   gchar *dimension, *endptr;
-  gint pt, width, height;
+  gint width, height;
   GstStructure *d;
 
-  /* <payload type number> <width>-<height> */
-
-  pt = g_ascii_strtoull (media_attr, &endptr, 10);
-  if (pt != GST_RTP_PAYLOAD_H263) {
-    GST_ERROR_OBJECT (filter,
-        "Framesize media attribute has incorrect payload type");
-    return FALSE;
-  } else if (*endptr != ' ') {
-    GST_ERROR_OBJECT (filter,
-        "Framesize media attribute has invalid payload type separator");
-    return FALSE;
-  }
-
-  dimension = endptr + 1;
-  width = g_ascii_strtoull (dimension, &endptr, 10);
+  width = g_ascii_strtoull (media_attr, &endptr, 10);
   if (width <= 0) {
     GST_ERROR_OBJECT (filter,
         "Framesize media attribute width out of valid range");