<p>Vadim Yanitskiy has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/13975">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">libmsc/msc_ho.c: fix unreacheable check of MSC-T role allocation<br><br>Change-Id: I46fa37ff27e8a4576fdc8edad894ee16759a6e7a<br>Fixes: CID#198413<br>---<br>M src/libmsc/msc_ho.c<br>1 file changed, 8 insertions(+), 7 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/75/13975/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libmsc/msc_ho.c b/src/libmsc/msc_ho.c</span><br><span>index 3a4258c..615b8cd 100644</span><br><span>--- a/src/libmsc/msc_ho.c</span><br><span>+++ b/src/libmsc/msc_ho.c</span><br><span>@@ -463,9 +463,8 @@</span><br><span>                               msc_i->ran_conn->ran_peer->fi->id, rp->fi->id);</span><br><span>             }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-           msc_t_alloc(msc_a->c.msub, rp);</span><br><span style="color: hsl(0, 100%, 40%);">-              msc_ho_send_handover_request(msc_a);</span><br><span style="color: hsl(0, 100%, 40%);">-            return;</span><br><span style="color: hsl(120, 100%, 40%);">+               msc_t = msc_t_alloc(msc_a->c.msub, rp);</span><br><span style="color: hsl(120, 100%, 40%);">+            break;</span><br><span> </span><br><span>   case MSC_NEIGHBOR_TYPE_REMOTE_MSC:</span><br><span>           ipa_name = msc_a->ho.new_cell.msc_ipa_name;</span><br><span>@@ -482,22 +481,24 @@</span><br><span>                              osmo_quote_str(ipa_name, -1));</span><br><span>                }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-           msc_t_remote_alloc(msc_a->c.msub, msc_a->c.ran, (const uint8_t*)ipa_name, strlen(ipa_name));</span><br><span style="color: hsl(0, 100%, 40%);">-              msc_ho_send_handover_request(msc_a);</span><br><span style="color: hsl(0, 100%, 40%);">-            return;</span><br><span style="color: hsl(120, 100%, 40%);">+               msc_t = msc_t_remote_alloc(msc_a->c.msub, msc_a->c.ran,</span><br><span style="color: hsl(120, 100%, 40%);">+                                    (const uint8_t *) ipa_name,</span><br><span style="color: hsl(120, 100%, 40%);">+                                           strlen(ipa_name));</span><br><span style="color: hsl(120, 100%, 40%);">+         break;</span><br><span> </span><br><span>   default:</span><br><span>             msc_ho_try_next_cell(msc_a, "unknown Handover target type %d\n", msc_a->ho.new_cell.type);</span><br><span>              return;</span><br><span>      }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   msc_t = msc_a_msc_t(msc_a);</span><br><span>  if (!msc_t) {</span><br><span>                /* There should definitely be one now. */</span><br><span>            msc_ho_failed(msc_a, GSM0808_CAUSE_EQUIPMENT_FAILURE,</span><br><span>                              "Cannot initiate Handover Request, failed to set up a target MSC-T\n");</span><br><span>              return;</span><br><span>      }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   msc_ho_send_handover_request(msc_a);</span><br><span> }</span><br><span> </span><br><span> static void msc_ho_rx_request_ack(struct msc_a *msc_a, struct msc_a_ran_dec_data *hra);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/13975">change 13975</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/13975"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I46fa37ff27e8a4576fdc8edad894ee16759a6e7a </div>
<div style="display:none"> Gerrit-Change-Number: 13975 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>