]> git.ipfire.org Git - thirdparty/asterisk.git/commit
chan_dahdi/sig_analog: Fix distinctive ring detection to suck less.
authorRichard Mudgett <rmudgett@digium.com>
Fri, 6 Mar 2015 19:52:04 +0000 (19:52 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Fri, 6 Mar 2015 19:52:04 +0000 (19:52 +0000)
commit3e6e5a977eb423b2dd163d55d4b371c9e9c80807
treee1961b0abff64193f3feeb478fa5b0b5a3ffa5e8
parent2befe82721e5f19b1dadfd2011cc491c25f15953
chan_dahdi/sig_analog: Fix distinctive ring detection to suck less.

The distinctive ring feature interferes with detecting Caller ID and
appears to have been broken for years.  What happens is if you have a
ring-ring cadence as used in the UK you get too many DAHDI events for the
distinctive ring pattern array and Caller ID detection is aborted.  I
think when Zapata/DAHDI added the ring begin event it broke distinctive
ring.  More events happen than before and the code does no filtering of
which event times are recorded in the pattern array.

* Made distinctive ring only record the ringt count when the ring ends
instead of on just any DAHDI event.  Distinctive ring can be ring,
ring-ring, ring-ring-ring, or different ring durations for the up to three
rings.

* Fixed the distinctive ring detection enable (chan_dahdi.conf option
usedistinctiveringdetection) to be per port instead of somewhat per port
and somewhat global.  This has been broken since v1.8.

* Fixed using the default distinctive ring context when the detected
pattern does not match any configured dringX patterns.  The default
context did not get set when the previous call was a matched distinctive
ring pattern and the current call is not matched.  This has been broken
since v1.8.

* Made distinctive ring have no effect on Caller ID detection when it is
disabled.  Caller ID detection just monitors for 10 seconds before giving
up.

* Fixed leak of struct callerid_state memory when a polarity reversal
during Caller ID detection causes the incoming call to be aborted.

DAHDI-1143
AST-1545
ASTERISK-24825 #close
Reported by: Richard Mudgett

ASTERISK-17588
Reported by: Daniel Flounders

Review: https://reviewboard.asterisk.org/r/4444/

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@432530 65c4cc65-6c06-0410-ace0-fbb531ad65f3
UPGRADE.txt
channels/chan_dahdi.c
channels/sig_analog.c
channels/sig_analog.h