]> git.ipfire.org Git - thirdparty/asterisk.git/commit
res_tonedetect: Add option for TONE_DETECT detection to auto stop.
authorNaveen Albert <asterisk@phreaknet.org>
Thu, 28 Aug 2025 12:29:02 +0000 (08:29 -0400)
committerGeorge Joseph <gjoseph@sangoma.com>
Wed, 17 Sep 2025 19:36:51 +0000 (13:36 -0600)
commit86a8f01677de119100139252762a5d7302b45464
tree6afc0dd4cb881b210ad56c07609ee078d3d28262
parente8539151c27f9b9d93510c905261ad5ad2ceb0db
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 6606fe8efebe4e85e6d63bcc71f6093faad79b30)
res/res_tonedetect.c