]> git.ipfire.org Git - thirdparty/asterisk.git/commit
bridge_softmix: Always remove audio from mixed frame.
authorJoshua C. Colp <jcolp@sangoma.com>
Tue, 19 May 2020 09:18:58 +0000 (06:18 -0300)
committerFriendly Automation <jenkins2@gerrit.asterisk.org>
Wed, 20 May 2020 14:15:37 +0000 (09:15 -0500)
commit17a878dd9945e1b395d5d93b2ad88303713e44cc
treeecfdd38559b5c9e9674defb676de6be042b3c1ee
parent4fba0f678b3181613a6f580f21e0d74c963c8ab0
bridge_softmix: Always remove audio from mixed frame.

When receiving audio from a channel we determine if it
is talking or silence based on a threshold value. If
this threshold is met we always mix the audio into the
conference bridge. If this threshold is not met we also
mix the audio into the conference bridge UNLESS the
drop silence option is enabled.

The code that removed the audio from the mixed frame
assumed that it was always not present if it did not
meet the threshold to be considered talking. This is
incorrect. If it has been stated that the audio was
mixed into the mixed frame then it has been mixed into
the mixed frame. By not removing audio that was
considered non-talking it was possible for a channel
to receive a slight echo of audio of itself at times.

This change ensures that the audio is always removed
from the mixed frame going back to the channel so it
no longer receives the slight echo.

ASTERISK-28898

Change-Id: I7b1b582cc1bcdb318ecc60c9d2e3d87ae31d55cb
bridges/bridge_softmix.c