Changeset 1217 for pjproject/branches/split-3rd-party/pjsip/src/pjsip/sip_msg.c

Timestamp:

Apr 28, 2007 1:00:33 PM (17 years ago)

Author:

bennylp

Message:

Fixed ticket #238: allow colon in Via parameters

File:

• pjproject/branches/split-3rd-party/pjsip/src/pjsip/sip_msg.c (modified) (1 diff)

pjproject/branches/split-3rd-party/pjsip/src/pjsip/sip_msg.c

@@ -1726 +1726 @@
     copy_advance_pair(buf, ";received=", 10, hdr->recvd_param);
     copy_advance_pair(buf, ";branch=", 8, hdr->branch_param);
-    
+
+    /* Via's via-extension params should be printed with token spec
+     * according to RFC 3261's generic-param production, but some
+     * endpoint uses pname/pvalue production for these params, by
+     * sending to us Via parameter containing ":".
+     * 
+     * So for interoperability sake, lets allow ":" in Via param.
+
     printed = pjsip_param_print_on(&hdr->other_param, buf, endbuf-buf, 
                                    &pjsip_TOKEN_SPEC,
-                                   &pjsip_TOKEN_SPEC, ';');
+                                   &pjsip_TOKEN_SPEC, ';');
+     */
+    printed = pjsip_param_print_on(&hdr->other_param, buf, endbuf-buf, 
+                                   &pjsip_PARAM_CHAR_SPEC,
+                                   &pjsip_PARAM_CHAR_SPEC, ';');
+
     if (printed < 0)
         return -1;