chan_sip: Make progressinband=never really mean 'never'
authorMatthew Jordan <mjordan@digium.com>
Thu, 17 Jul 2014 21:04:01 +0000 (21:04 +0000)
committerMatthew Jordan <mjordan@digium.com>
Thu, 17 Jul 2014 21:04:01 +0000 (21:04 +0000)
commit1ce23d4534994fdd8bfb8ad3b9ca1884194097be
tree651d4b7db01db8141baeddb9da8639875bc36a80
parent96e5e491fa800316b1c399b730cb3d320e6eeeaa
chan_sip: Make progressinband=never really mean 'never'

progressinband=never in sip.conf is easily defeated if an onward trunk sends a
progress indication of its own. This is almost certain to happen if the onward
trunk is ISDN or IAX as these technologies send a progress indication even if
early media is not required. This progress message is passed to the caller,
and causes the "never" option to be rather badly named.

This patch changes the behaviour of this setting in the following ways:

1) In sip_write(), do not pass the media unless we have either progressed
   beyond INV_EARLY_MEDIA, or we are in INV_EARLY_MEDIA state, and early
   media is both set-up and wanted. This helps resolve double-ringing on some
   buggy handsets.

2) In sip_indicate(), if we see AST_CONTROL_PROGRESS, but
   SIP_PROG_INBAND_NEVER is set, send a 180 Ringing instead to avoid implicitly
   enabling early media. Avoid sending double ring indications.

NOTE: the meaning of the SIP_PROGRESS_SENT flag changes slightly in this patch
to also encapsulate the fact that a channel has *sent or received* a 183
Progress indication. This makes the updated code in sip_write() much more
simple.

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

ASTERISK-23972 #close
Reported by: Steve Davies
patches:
  inband_never_present_early_media2 uploaded by Steve Davies (License 5012)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@418868 65c4cc65-6c06-0410-ace0-fbb531ad65f3
UPGRADE.txt
channels/chan_sip.c