]> git.ipfire.org Git - thirdparty/asterisk.git/commit
res_tonedetect: Add option for TONE_DETECT detection to auto stop.
authorNaveen Albert <asterisk@phreaknet.org>
Fri, 22 Aug 2025 15:51:15 +0000 (11:51 -0400)
committerGeorge Joseph <gjoseph@sangoma.com>
Wed, 17 Sep 2025 17:40:38 +0000 (11:40 -0600)
commiteab86427114e2bee926fe16e00f25ba4250bce9f
tree524d6d36ed4595679d9077a1b5e44b8b1c4a3c4b
parent2d2151c0519e9fb85eab3524f8d848d9ff581298
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