mediaproxy: handle return pointer values

kamailio · Jul 23, 2017 · 65ac6de · 65ac6de
1 parent 0efa76d
commit 65ac6de
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/modules/mediaproxy/mediaproxy.c b/src/modules/mediaproxy/mediaproxy.c
@@ -809,12 +809,14 @@ static str
 get_sdp_line_separator(str *sdp)
 {
     char *ptr, *end_ptr, *sdp_end;
-    str separator;
+    str separator = STR_NULL;
 
     sdp_end = sdp->s + sdp->len;
 
     ptr = find_line_starting_with(sdp, "v=", False);
+    if(ptr==NULL) { return separator; }
     end_ptr = findendline(ptr, sdp_end-ptr);
+    if(end_ptr==NULL) { return separator; }
     separator.s = ptr = end_ptr;
     while ((*ptr=='\n' || *ptr=='\r') && ptr<sdp_end)
         ptr++;
@@ -1329,7 +1331,7 @@ insert_element(struct sip_msg *msg, char *position, char *element)
 
     len = strlen(element);
 
-    buf = pkg_malloc(len);
+    buf = pkg_malloc(len+1);
     if (!buf) {
         LM_ERR("out of memory\n");
         return False;
@@ -1343,6 +1345,7 @@ insert_element(struct sip_msg *msg, char *position, char *element)
     }
 
     memcpy(buf, element, len);
+    buf[len] = '\0';
 
     if (insert_new_lump_after(anchor, buf, len, 0)==0) {
         LM_ERR("failed to insert new element\n");