From 5b8db4e4227aefdd45ca936b8c4240cb29b827f6 Mon Sep 17 00:00:00 2001 From: Torrey Searle Date: Mon, 25 Feb 2019 16:41:44 +0100 Subject: [PATCH] res/res_rtp_asterisk: smoother can cause wrong timestamps if dtmf happen Delivery timeval in the smoother object will fall behind while a DTMF is being generated. This can eventually lead to invalid rtp timestamps. To prevent this from happening the smoother needs to be reset after every DTMF to keep the timing up to date. ASTERISK-28303 #close Change-Id: Iaba3f7b428ebd72a4caa90e13b829ab4f088310f --- res/res_rtp_asterisk.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c index c2064972e2..84b2088455 100644 --- a/res/res_rtp_asterisk.c +++ b/res/res_rtp_asterisk.c @@ -3957,6 +3957,12 @@ static int ast_rtp_dtmf_end_with_duration(struct ast_rtp_instance *instance, cha /* Oh and we can't forget to turn off the stuff that says we are sending DTMF */ rtp->lastts += calc_txstamp(rtp, NULL) * DTMF_SAMPLE_RATE_MS; + + /* Reset the smoother as the delivery time stored in it is now out of date */ + if (rtp->smoother) { + ast_smoother_free(rtp->smoother); + rtp->smoother = NULL; + } cleanup: rtp->sending_digit = 0; rtp->send_digit = 0; -- 2.47.2