[asterisk-commits] jrose: trunk r399006 - in /trunk: ./ channels/ channels/sip/include/

[SVN commits to the Asterisk project]

Author: jrose
Date: Thu Sep 12 15:27:56 2013
New Revision: 399006

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=399006
Log:
chan_sip: Revert r398835 due to failing tests involving originate

(issue ASTERISK-22424)
Reported by: Jonathan Rose
........

Merged revisions 398977 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 398986 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 398991 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    trunk/   (props changed)
    trunk/channels/chan_sip.c
    trunk/channels/sip/include/sip.h

Propchange: trunk/
------------------------------------------------------------------------------
--- branch-12-merged (original)
+++ branch-12-merged Thu Sep 12 15:27:56 2013
@@ -1,1 +1,1 @@
-/branches/12:1-398558,398560-398577,398579-398927,398938
+/branches/12:1-398558,398560-398577,398579-398927,398938,398991

Modified: trunk/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_sip.c?view=diff&rev=399006&r1=399005&r2=399006
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Thu Sep 12 15:27:56 2013
@@ -23250,15 +23250,6 @@
 					ast_set_flag(&p->flags[0], SIP_PENDINGBYE);
 				}
 			ast_rtp_instance_activate(p->rtp);
-		} else if (!reinvite) {
-			struct ast_sockaddr remote_address = {{0,}};
-
-			ast_rtp_instance_get_remote_address(p->rtp, &remote_address);
-			if (ast_sockaddr_isnull(&remote_address) || (!ast_strlen_zero(p->theirprovtag) && strcmp(p->theirtag, p->theirprovtag))) {
-				ast_log(LOG_WARNING, "Received response: \"200 OK\" from '%s' without SDP\n", p->relatedpeer->name);
-				ast_set_flag(&p->flags[0], SIP_PENDINGBYE);
-				ast_rtp_instance_activate(p->rtp);
-			}
 		}
 
 		if (!req->ignore && p->owner) {
@@ -24209,11 +24200,7 @@
 
 		gettag(req, "To", tag, sizeof(tag));
 		ast_string_field_set(p, theirtag, tag);
-	} else {
-		/* Store theirtag to track for changes when 200 responses to invites are received without SDP */
-		ast_string_field_set(p, theirprovtag, p->theirtag);
-	}
-
+	}
 	/* This needs to be configurable on a channel/peer level,
 	   not mandatory for all communication. Sadly enough, NAT implementations
 	   are not so stable so we can always rely on these headers.

Modified: trunk/channels/sip/include/sip.h
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/sip/include/sip.h?view=diff&rev=399006&r1=399005&r2=399006
==============================================================================
--- trunk/channels/sip/include/sip.h (original)
+++ trunk/channels/sip/include/sip.h Thu Sep 12 15:27:56 2013
@@ -1019,7 +1019,6 @@
 		AST_STRING_FIELD(rdnis);        /*!< Referring DNIS */
 		AST_STRING_FIELD(redircause);   /*!< Referring cause */
 		AST_STRING_FIELD(theirtag);     /*!< Their tag */
-		AST_STRING_FIELD(theirprovtag); /*!< Provisional their tag, used when evaluating responses to invites */
 		AST_STRING_FIELD(tag);          /*!< Our tag for this session */
 		AST_STRING_FIELD(username);     /*!< [user] name */
 		AST_STRING_FIELD(peername);     /*!< [peer] name, not set if [user] */