From eab86427114e2bee926fe16e00f25ba4250bce9f Mon Sep 17 00:00:00 2001 From: Naveen Albert Date: Fri, 22 Aug 2025 11:51:15 -0400 Subject: [PATCH] res_tonedetect: Add option for TONE_DETECT detection to auto stop. One of the problems with TONE_DETECT as it was originally written is that if a tone is detected multiple times, it can trigger the redirect logic multiple times as well. For example, if we do an async goto in the dialplan after detecting a tone, because the detector is still active until explicitly disabled, if we detect the tone again, we will branch again and start executing that dialplan a second time. This is rarely ever desired behavior, and can happen if the detector is not removed quickly enough. Add a new option, 'e', which automatically disables the detector once the desired number of matches have been heard. This eliminates the potential race condition where previously the detector would need to be disabled immediately, but doing so quickly enough was not guaranteed. This also allows match criteria to be retained longer if needed, so the detector does not need to be destroyed prematurely. Resolves: #1390 UserNote: The 'e' option for TONE_DETECT now allows detection to be disabled automatically once the desired number of matches have been fulfilled, which can help prevent race conditions in the dialplan, since TONE_DETECT does not need to be disabled after a hit. (cherry picked from commit 016a53beba5884d3ed4c96837816874e74efa44f) --- res/res_tonedetect.c | 32 ++++++++++++++++++++++++-------- 1 file changed, 24 insertions(+), 8 deletions(-) diff --git a/res/res_tonedetect.c b/res/res_tonedetect.c index 1ec36932c2..99dd40538c 100644 --- a/res/res_tonedetect.c +++ b/res/res_tonedetect.c @@ -214,13 +214,16 @@ +