[asterisk-commits] dvossel: branch dvossel/sip_forked_responses r289129 - /team/dvossel/sip_fork...

[SVN commits to the Asterisk project]

Author: dvossel
Date: Tue Sep 28 13:24:01 2010
New Revision: 289129

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=289129
Log:
Fixed some naming confusion. This branch handles forked outgoing requests.

Modified:
    team/dvossel/sip_forked_responses/channels/chan_sip.c

Modified: team/dvossel/sip_forked_responses/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/team/dvossel/sip_forked_responses/channels/chan_sip.c?view=diff&rev=289129&r1=289128&r2=289129
==============================================================================
--- team/dvossel/sip_forked_responses/channels/chan_sip.c (original)
+++ team/dvossel/sip_forked_responses/channels/chan_sip.c Tue Sep 28 13:24:01 2010
@@ -7256,8 +7256,8 @@
 enum match_req_res {
 	SIP_REQ_MATCH,
 	SIP_REQ_NOT_MATCH,
-	SIP_REQ_LOOP_DETECTED,
-	SIP_REQ_FORKED_RESPONSE,
+	SIP_REQ_LOOP_DETECTED,  /* multiple incoming requests containing different branch parameters have been detected */
+	SIP_REQ_FORKED_REQUEST, /* an outgoing request has been forked resulting in multiple separate 200ok responses */
 };
 
 /*
@@ -7316,7 +7316,7 @@
 					!ast_strlen_zero(invite_branch) &&
 					!ast_strlen_zero(arg->viabranch) &&
 					!strcmp(invite_branch, arg->viabranch)) {
-					return SIP_REQ_FORKED_RESPONSE;
+					return SIP_REQ_FORKED_REQUEST;
 				}
 
 				/* The totag did not match the one we had stored, and this is not a Forked Request. */
@@ -7537,7 +7537,7 @@
 				ao2_iterator_destroy(iterator);
 				return NULL;
 			case SIP_REQ_FORKED_REQUEST:
-			//todohere handle forked request
+			//todohere handle forked response
 			case SIP_REQ_NOT_MATCH:
 			default:
 				dialog_unref(sip_pvt_ptr, "pvt did not match incoming SIP msg, unref from search");