<p>Alexander Traud <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/15379">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">chan_sip: Set up calls without audio (text+video), again.<br><br>The previous commit 6d980de fixed this issue in the core of Asterisk.<br>With that, each channel technology can be used without audio<br>theoretically. Practically, the channel-technology driver chan_sip<br>turned out to have an invalid check preventing that. chan_sip tested<br>whether there is at least one audio format. However, chan_sip has to<br>test whether there is at least one format. More cannot be tested while<br>requesting chan_sip because only the [general] capabilities but not the<br>[peer] caps are known yet. And the [peer] caps might not be a subset or<br>show any intersection with the [general] caps. This change here fixes<br>this.<br><br>The original commit f04d5fb, thirteen years ago, contained a software<br>bug as it passed ANY audio capability to the channel-technology driver.<br>Instead, it should have passed NO audio format. Therefore, this<br>addressed issue here was not noticed in Asterisk 1.6.x and Asterisk 1.8.<br>Then, Asterisk 10 changed that from ANY to NO, but nobody reported since<br>then.<br><br>ASTERISK-29265<br><br>Change-Id: Ic16a3bf13cd1b5c4fc4041ed74961177d96b600f<br>---<br>M channels/chan_sip.c<br>1 file changed, 2 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/79/15379/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15379">change 15379</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/15379"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ic16a3bf13cd1b5c4fc4041ed74961177d96b600f </div>
<div style="display:none"> Gerrit-Change-Number: 15379 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>