<p>Joshua Colp has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/6030">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bridge_softmix: Don't reorder streams on participant leaving.<br><br>When a participant leaves a bridge while operating in SFU mode<br>their respective stream on every other participant needs to be<br>removed. Leaving the stream out of the new topology results in<br>every stream after it being moved and reordered. This causes<br>problems with clients. Instead simply mark the stream as removed<br>which leaves it in place in the SDP and doesn't reorder or touch<br>any other streams.<br><br>ASTERISK-27136<br><br>Change-Id: I4b3f840adcdf69b83842b0d8a737665ba0ef9cb1<br>---<br>M bridges/bridge_softmix.c<br>1 file changed, 5 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/30/6030/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/bridges/bridge_softmix.c b/bridges/bridge_softmix.c<br>index ae877eb..0b1a6f0 100644<br>--- a/bridges/bridge_softmix.c<br>+++ b/bridges/bridge_softmix.c<br>@@ -701,14 +701,15 @@<br> <br>           stream = ast_stream_topology_get_stream(source, i);<br> <br>-               if (is_video_dest(stream, channel_name, NULL)) {<br>-                     continue;<br>-            }<br>-<br>          stream_clone = ast_stream_clone(stream, NULL);<br>                if (!stream_clone) {<br>                  continue;<br>             }<br>+<br>+         if (is_video_dest(stream, channel_name, NULL)) {<br>+                     ast_stream_set_state(stream, AST_STREAM_STATE_REMOVED);<br>+              }<br>+<br>          if (ast_stream_topology_append_stream(dest, stream_clone) < 0) {<br>                   ast_stream_free(stream_clone);<br>                }<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6030">change 6030</a>. To unsubscribe, 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/6030"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I4b3f840adcdf69b83842b0d8a737665ba0ef9cb1 </div>
<div style="display:none"> Gerrit-Change-Number: 6030 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Joshua Colp <jcolp@digium.com> </div>