sysmocom
/
asterisk
11
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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
This commit is contained in:
Torrey Searle 2019-02-25 16:41:44 +01:00 committed by Torrey Searle
parent aefc501913
commit 5b8db4e422
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
res/res_rtp_asterisk.c
View File

@ -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;