From: Kevin Harwell <kharwell@digium.com>
Date: Fri, 4 Nov 2016 15:57:43 +0000 (-0500)
Subject: Revert "chan_sip: Fix lastrtprx always updated"
X-Git-Tag: 13.13.0-rc1~48
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cb30963d222cb1e12af9bbf8dfed58001c9fcaf4;p=thirdparty%2Fasterisk.git

Revert "chan_sip: Fix lastrtprx always updated"

This reverts commit 93332cb1d0eea18021ea6538237297e627d6e2fc.

Unfortunately, the aforementioned commit caused a regression (incoming calls
would eventually disconnect). Thus it is being removed.

ASTERISK-26523 #close
ASTERISK-25270

Change-Id: Ibf5586adc303073a8eac667a4cbfdb6be184a64d
---

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index b6b4cc86e5..b661f0de22 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -8628,9 +8628,7 @@ static struct ast_frame *sip_read(struct ast_channel *ast)
 
 	sip_pvt_lock(p);
 	fr = sip_rtp_read(ast, p, &faxdetected);
-	if (fr && fr->frametype != AST_FRAME_NULL) {
-		p->lastrtprx = time(NULL);
-	}
+	p->lastrtprx = time(NULL);
 
 	/* If we detect a CNG tone and fax detection is enabled then send us off to the fax extension */
 	if (faxdetected && ast_test_flag(&p->flags[1], SIP_PAGE2_FAX_DETECT_CNG)) {