[asterisk-commits] rizzo: trunk r46035 - /trunk/channels/chan_sip.c

asterisk-commits at lists.digium.com asterisk-commits at lists.digium.com
Mon Oct 23 13:01:15 MST 2006


Author: rizzo
Date: Mon Oct 23 15:01:14 2006
New Revision: 46035

URL: http://svn.digium.com/view/asterisk?rev=46035&view=rev
Log:
remove a useless check for ocseq = 0.
As discussed on the mailing lists, 0 is a legal value
for Cseq, so there is no point to treat it specially.


Modified:
    trunk/channels/chan_sip.c

Modified: trunk/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_sip.c?rev=46035&r1=46034&r2=46035&view=diff
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Mon Oct 23 15:01:14 2006
@@ -14421,11 +14421,11 @@
 		   a request and have a response, or at least get a response
 		   within an existing dialog */
 		/* Response to our request -- Do some sanity checks */	
-		if (p->ocseq && (p->ocseq < seqno)) {
+		if (p->ocseq < seqno) {
 			if (option_debug)
 				ast_log(LOG_DEBUG, "Ignoring out of order response %d (expecting %d)\n", seqno, p->ocseq);
 			return -1;
-		} else if (p->ocseq && (p->ocseq != seqno)) {
+		} else if (p->ocseq != seqno) {
 			/* ignore means "don't do anything with it" but still have to 
 			   respond appropriately  */
 			ast_set_flag(req, SIP_PKT_IGNORE);



More information about the asterisk-commits mailing list