Changeset 4613 for pjproject/trunk/pjmedia/src/pjmedia/endpoint.c

Timestamp:

Oct 8, 2013 9:08:13 AM (11 years ago)

Author:

bennylp

Message:

Re #1703: fixing general bugs. First installment: correct handling of snprintf return value

File:

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

pjproject/trunk/pjmedia/src/pjmedia/endpoint.c

@@ -491 +491 @@
 
             /* Print codec PT */
-            buf_len += pj_ansi_snprintf(buf, 
-                                        MAX_FMTP_STR_LEN - buf_len, 
-                                        "%d", 
+            buf_len += pj_ansi_snprintf(buf,
+                                        MAX_FMTP_STR_LEN - buf_len,
+                                        "%d",
                                         codec_info->pt);
 
@@ -501 +501 @@
                 /* Check if buf still available */
                 test_len = dec_fmtp->param[i].val.slen + 
-                           dec_fmtp->param[i].name.slen;
+                           dec_fmtp->param[i].name.slen + 2;
                 if (test_len + buf_len >= MAX_FMTP_STR_LEN)
                     return PJ_ETOOBIG;
@@ -687 +687 @@
                 /* Check if buf still available */
                 test_len = dec_fmtp->param[j].val.slen + 
-                           dec_fmtp->param[j].name.slen;
+                           dec_fmtp->param[j].name.slen + 2;
                 if (test_len + buf_len >= MAX_FMTP_STR_LEN)
                     return PJ_ETOOBIG;