]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Local channels: Do not block control -1 payloads. 49/949/1
authorMark Michelson <mmichelson@digium.com>
Wed, 22 Jul 2015 14:46:29 +0000 (09:46 -0500)
committerMark Michelson <mmichelson@digium.com>
Wed, 22 Jul 2015 14:46:29 +0000 (09:46 -0500)
commit525bbf7d4eab8f9953bff26388a0a41c3bf2939a
tree254a4c252f44712c538069b04f8efa63511815a8
parent43a52df20d92de81604ce3c0dc9231d0e75eef03
Local channels: Do not block control -1 payloads.

Control frames with a -1 payload are used as a special signal to stop
playtones generators on channels. This indication is sent both by
app_dial as well as by ast_answer() when a call is answered in case any
tones were being generated on a calling channel.

This control frame type was made to stop traversing local channel pairs
as an optimization, because it was thought that it was unnecessary to
send these indications, and allowing such unnecessary control frames to
traverse the local channels would cause the local channels to optimize
away less quickly.

As it turns out, through some special magic dialplan code, it is
possible to have a tones being played on a non-local channel, and it is
important for the local channel to convey that the tones should be
stopped. The result of having tones continue to be played on the
non-local channel is that the tones play even once the channel has been
bridged. By not blocking the -1 control frame type, we can ensure that
this situation does not happen.

ASTERISK-25250 #close
Reported by Etienne Lessard

Change-Id: I40227e4249d6d61dc6a9b08bbe9ee3aa18be7e30
channels/chan_local.c