Changeset 199 for pjproject/trunk/pjmedia/src/pjmedia/sdp.c

Timestamp:

Feb 19, 2006 3:35:54 PM (18 years ago)

Author:

bennylp

Message:

Fixed packing error in rtp header, and sdp validation supports non numeric pt for broken uas

File:

• pjproject/trunk/pjmedia/src/pjmedia/sdp.c (modified) (1 diff)

pjproject/trunk/pjmedia/src/pjmedia/sdp.c

@@ -1149 +1149 @@
         /* Verify payload type. */
         for (j=0; j<m->desc.fmt_count; ++j) {
-            unsigned pt = pj_strtoul(&m->desc.fmt[j]);
-
-            /* Payload type is between 0 and 127. */
-            CHECK( pt <= 127, PJMEDIA_SDP_EINPT);
-
-            /* If port is not zero, then for each dynamic payload type, an
-             * rtpmap attribute must be specified.
+
+            /* Arrgh noo!! Payload type can be non-numeric!!
+             * RTC based programs sends "null" for instant messaging!
              */
-            if (m->desc.port != 0 && pt >= 96) {
-                const pjmedia_sdp_attr *a;
-
-                a = pjmedia_sdp_media_find_attr(m,&STR_RTPMAP,&m->desc.fmt[j]);
-                CHECK( a != NULL, PJMEDIA_SDP_EMISSINGRTPMAP);
+            if (pj_isdigit(*m->desc.fmt[j].ptr)) {
+                unsigned pt = pj_strtoul(&m->desc.fmt[j]);
+
+                /* Payload type is between 0 and 127. 
+                 */
+                CHECK( pt <= 127, PJMEDIA_SDP_EINPT);
+
+                /* If port is not zero, then for each dynamic payload type, an
+                 * rtpmap attribute must be specified.
+                 */
+                if (m->desc.port != 0 && pt >= 96) {
+                    const pjmedia_sdp_attr *a;
+
+                    a = pjmedia_sdp_media_find_attr(m, &STR_RTPMAP, 
+                                                    &m->desc.fmt[j]);
+                    CHECK( a != NULL, PJMEDIA_SDP_EMISSINGRTPMAP);
+                }
             }
         }