Changeset 1635 for pjproject/trunk

Timestamp:

Dec 17, 2007 12:16:08 PM (17 years ago)

Author:

bennylp

Message:

Fixed errors in SDP: 1) scanner is not initialized before pjmedia_sdp_parse() is called, 2) added null termination in attribute for more robustness

File:

• pjproject/trunk/pjmedia/src/pjmedia/sdp.c (modified) (3 diffs)

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

@@ -249 +249 @@
     PJ_ASSERT_RETURN(attr->value.slen != 0, PJMEDIA_SDP_EINATTR);
 
+    init_sdp_parser();
+
     /* Check if input is null terminated, and null terminate if
      * necessary. Unfortunately this may crash the application if
@@ -367 +369 @@
     PJ_ASSERT_RETURN(pj_strcmp2(&attr->name, "rtcp")==0, PJ_EINVALIDOP);
 
+    init_sdp_parser();
+
     /* fmtp BNF:
      *  a=rtcp:<port> [nettype addrtype address]
@@ -464 +468 @@
     attr->value.slen = len;
     attr->value.ptr = (char*) pj_pool_alloc(pool, attr->value.slen);
-    pj_memcpy(attr->value.ptr, tempbuf, attr->value.slen);
+    pj_memcpy(attr->value.ptr, tempbuf, attr->value.slen+1);
 
     *p_attr = attr;