[asterisk-commits] twilson: branch 10 r369580 - in /branches/10: ./ channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: twilson
Date: Tue Jul  3 12:02:18 2012
New Revision: 369580

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=369580
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)
........

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

Modified:
    branches/10/   (props changed)
    branches/10/channels/chan_sip.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/channels/chan_sip.c?view=diff&rev=369580&r1=369579&r2=369580
==============================================================================
--- branches/10/channels/chan_sip.c (original)
+++ branches/10/channels/chan_sip.c Tue Jul  3 12:02:18 2012
@@ -20742,9 +20742,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);
 		}
 	}