]> git.ipfire.org Git - thirdparty/asterisk.git/commit
chan_sip: Set up calls without audio (text+video), again.
authorAlexander Traud <pabstraud@compuserve.com>
Wed, 27 Jan 2021 17:42:06 +0000 (18:42 +0100)
committerAlexander Traud <pabstraud@compuserve.com>
Wed, 3 Feb 2021 09:01:12 +0000 (03:01 -0600)
commit87ad1138ffec1f87a765447999270b1bdacf535f
treeb76df9d99bf51e9ea88b589519cc0a5189c74bf9
parent088816284a51a72207344324671beb53ada56ae7
chan_sip: Set up calls without audio (text+video), again.

The previous commit 6d980de fixed this issue in the core of Asterisk.
With that, each channel technology can be used without audio
theoretically. Practically, the channel-technology driver chan_sip
turned out to have an invalid check preventing that. chan_sip tested
whether there is at least one audio format. However, chan_sip has to
test whether there is at least one format. More cannot be tested while
requesting chan_sip because only the [general] capabilities but not the
[peer] caps are known yet. And the [peer] caps might not be a subset or
show any intersection with the [general] caps. This change here fixes
this.

The original commit f04d5fb, thirteen years ago, contained a software
bug as it passed ANY audio capability to the channel-technology driver.
Instead, it should have passed NO audio format. Therefore, this
addressed issue here was not noticed in Asterisk 1.6.x and Asterisk 1.8.
Then, Asterisk 10 changed that from ANY to NO, but nobody reported since
then.

ASTERISK-29265

Change-Id: Ic16a3bf13cd1b5c4fc4041ed74961177d96b600f
channels/chan_sip.c