<p>dexter <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/9231">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">MSC_ConnectionHandler: check channel mode in f_establish_fully<br><br>The helper function f_establish_fully() checks the channel type<br>for compatibility. If the channel type is compatible with the<br>desired mode a channel mode modification could be necessary if<br>the current channel mode is different from the desired channel<br>mode. However, this is not checked at the momemend. We just<br>blindly expect a MODE MODIFY message from the BSC and ignore<br>the cases where the current channel mode and the desired channel<br>mode already matches up. This is the case if only a signalling<br>channel is requested with f_establish_fully for example.<br><br>- Check if the channel mode of the current channel and the<br>  desired channel mode match up.<br><br>- Make sure that the MODE MODIFY from the BSC is only<br>  expected when the channel modes are different.<br><br>Note: The function f_channel_needs_modify() that is used<br>to determine if a channel modification is needed or not<br>does not cover all cases yet. (see fixme note)<br><br>Change-Id: I9004f299220b01ecea6b2316ba3f913c316947dc<br>Closes: OS#2762<br>Related: OS#2936<br>---<br>M bsc/MSC_ConnectionHandler.ttcn<br>1 file changed, 51 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/31/9231/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/9231">change 9231</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/9231"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I9004f299220b01ecea6b2316ba3f913c316947dc </div>
<div style="display:none"> Gerrit-Change-Number: 9231 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>