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

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/10@369580 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 689ed736dd83f82acc07462fcf847e1e1aedf9a7..4f34d13d9993604e43c6f674b30fbbb3dec1398e 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -20742,9 +20742,6 @@ static void handle_response_invite(struct sip_pvt *p, int resp, const char *rest
                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);
                }
        }