[asterisk-commits] twilson: branch 1.8 r369579 - /branches/1.8/channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: twilson
Date: Tue Jul  3 11:58:16 2012
New Revision: 369579

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=369579
Log:
More improvements to re-INVITEs timing out after a provisional response

There is no need to call check_pendings() on a final response to an INVITE
when destroying the scheduler entry as it will be done later during normal
processing.

(issue ASTERISK-19992)

Modified:
    branches/1.8/channels/chan_sip.c

Modified: branches/1.8/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/channels/chan_sip.c?view=diff&rev=369579&r1=369578&r2=369579
==============================================================================
--- branches/1.8/channels/chan_sip.c (original)
+++ branches/1.8/channels/chan_sip.c Tue Jul  3 11:58:16 2012
@@ -20122,9 +20122,6 @@
 		p->ongoing_reinvite = 0;
 		if (p->reinviteid > -1) {
 			AST_SCHED_DEL_UNREF(sched, p->reinviteid, dialog_unref(p, "unref dialog for reinvite timeout because of a final response"));
-			/* Since we got a final response to the reinvite, but were relying on the reinvite_timeout
-			 * function to clean up after the reinvite, we need to make sure and call check_pendings */
-			check_pendings(p);
 		}
 	}