Changeset 2915 for pjproject/trunk/pjsip/src/pjsip/sip_transaction.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_transaction.c (modified) (1 diff)

pjproject/trunk/pjsip/src/pjsip/sip_transaction.c

@@ -2841 +2841 @@
             /* Process incoming ACK request. */
 
+            /* Verify that this is an INVITE transaction */
+            if (tsx->method.id != PJSIP_INVITE_METHOD) {
+                PJ_LOG(2, (tsx->obj_name, 
+                           "Received illegal ACK for %.*s transaction",
+                           (int)tsx->method.name.slen,
+                           tsx->method.name.ptr));
+                return PJSIP_EINVALIDMETHOD;
+            }
+
             /* Cease retransmission. */
             if (tsx->retransmit_timer.id != 0) {