Changeset 2915 for pjproject/trunk/pjsip/src/pjsip/sip_transport.c

Timestamp:

Sep 22, 2009 5:56:44 PM (15 years ago)

Author:

bennylp

Message:

Fixed ticket #959: Assertion upon receiving malformed SIP messages (thanks Andrey Kovalenko for the report)

• transaction checks for the method before processing incoming ACK request
• transport layer checks the validity of status code in the response
• added SIPP scenario to reproduce the bad ACK request

File:

• pjproject/trunk/pjsip/src/pjsip/sip_transport.c (modified) (1 diff)

pjproject/trunk/pjsip/src/pjsip/sip_transport.c

@@ -1440 +1440 @@
                 rdata->msg_info.via->rport_param = rdata->pkt_info.src_port;
             }
+        } else {
+            /* Drop malformed responses */
+            if (rdata->msg_info.msg->line.status.code < 100 ||
+                rdata->msg_info.msg->line.status.code >= 700)
+            {
+                mgr->on_rx_msg(mgr->endpt, PJSIP_EINVALIDSTATUS, rdata);
+                goto finish_process_fragment;
+            }
         }