Ticket #832: ticket832.2.patch

pjsip/src/test/msg_test.c

@@ -365 +365 @@
     hdr2 = ref_msg->hdr.next;
 
     while (hdr1 != &parsed_msg->hdr && hdr2 != &ref_msg->hdr) {
-        len = hdr1->vptr->print_on(hdr1, str1.ptr, BUFLEN);
-        if (len < 1) {
+        len = pjsip_hdr_print_on(hdr1, str1.ptr, BUFLEN);
+        if (len < 0) {
             status = -40;
             goto on_return;
         }
         str1.ptr[len] = '\0';
         str1.slen = len;
 
-        len = hdr2->vptr->print_on(hdr2, str2.ptr, BUFLEN);
-        if (len < 1) {
+        len = pjsip_hdr_print_on(hdr2, str2.ptr, BUFLEN);
+        if (len < 0) {
             status = -50;
             goto on_return;
         }
@@ -1937 +1937 @@
         /* Print the parsed header*/
         output = (char*) pj_pool_alloc(pool, 1024);
         len = pjsip_hdr_print_on(parsed_hdr1, output, 1024);
-        if (len < 1 || len >= 1024) {
+        if (len < 0 || len >= 1024) {
             PJ_LOG(3,(THIS_FILE, "    header too long: %s: %s", test->hname, test->hcontent));
             return -530;
         }

pjsip/src/pjsip/sip_msg.c

@@ -456 +456 @@
 
     /* Print each of the headers. */
     for (hdr=msg->hdr.next; hdr!=&msg->hdr; hdr=hdr->next) {
-        len = (*hdr->vptr->print_on)(hdr, p, end-p);
-        if (len < 1)
+        len = pjsip_hdr_print_on(hdr, p, end-p);
+        if (len < 0)
             return -1;
-        p += len;
 
-        if (p+3 >= end)
-            return -1;
+        if (len > 0) {
+            p += len;
+            if (p+3 >= end)
+                return -1;
 
-        *p++ = '\r';
-        *p++ = '\n';
+            *p++ = '\r';
+            *p++ = '\n';
+        }
     }
 
     /* Process message body. */
@@ -1601 +1603 @@
     char *startbuf = buf;
     char *endbuf = buf + size;
     const pjsip_parser_const_t *pc = pjsip_parser_const();
+    pjsip_sip_uri *sip_uri;
+    pjsip_param *p;
+
+    /* Check the proprietary param 'hide', don't print this header 
+     * if it exists in the route URI.
+     */
+    sip_uri = (pjsip_sip_uri*) pjsip_uri_get_uri(hdr->name_addr.uri);
+    p = sip_uri->other_param.next;
+    while (p != &sip_uri->other_param) {
+        const pj_str_t st_hide = {"hide", 4};
+
+        if (pj_stricmp(&p->name, &st_hide) == 0) {
+            /* Check if param 'hide' is specified without 'lr'. */
+            pj_assert(sip_uri->lr_param != 0);
+            return 0;
+        }
+        p = p->next;
+    }
+
     /* Route and Record-Route don't compact forms */
 
     copy_advance(buf, hdr->name);

pjsip/src/pjsip/sip_ua_layer.c

@@ -907 +907 @@
     char userinfo[128];
 
     len = pjsip_hdr_print_on(dlg->remote.info, userinfo, sizeof(userinfo));
-    if (len < 1)
+    if (len < 0)
         pj_ansi_strcpy(userinfo, "<--uri too long-->");
     else
         userinfo[len] = '\0';

pjsip/src/pjsua-lib/pjsua_call.c

@@ -2694 +2694 @@
     /* Dump invite sesion info. */
 
     len = pjsip_hdr_print_on(dlg->remote.info, userinfo, sizeof(userinfo));
-    if (len < 1)
+    if (len < 0)
         pj_ansi_strcpy(userinfo, "<--uri too long-->");
     else
         userinfo[len] = '\0';

pjsip-apps/src/samples/siprtp_report.c

@@ -77 +77 @@
 
     /* Call identification */
     len = pjsip_hdr_print_on(dlg->remote.info, userinfo, sizeof(userinfo));
-    if (len < 1)
+    if (len < 0)
         pj_ansi_strcpy(userinfo, "<--uri too long-->");
     else
         userinfo[len] = '\0';