<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/11275">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">msc_mgcp: move mncc struct initalization to where its actually needed<br><br>The function _handle_error() initalizes a struct gsm_mncc variable<br>on startup. The initalization accesses mgcp_ctx->trans->callref. All<br>this is done before the assertion on mgcp_ctx. Later in the code one<br>finds an if which tests on mgcp_ctx->free_ctx. This is the only part of<br>the code that accesses the mncc struct variable. We should move the<br>initalization there as well.<br><br>- Move initalization of struct gsm_mncc mncc into the if body<br>  that uses it.<br><br>Change-Id: I86983eabd999c4275dcc0e4a169ef2aa1e33c747<br>Related: OS#3635<br>---<br>M src/libmsc/msc_mgcp.c<br>1 file changed, 11 insertions(+), 9 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libmsc/msc_mgcp.c b/src/libmsc/msc_mgcp.c</span><br><span>index acdb785..c2bbe5f 100644</span><br><span>--- a/src/libmsc/msc_mgcp.c</span><br><span>+++ b/src/libmsc/msc_mgcp.c</span><br><span>@@ -158,15 +158,7 @@</span><br><span> {</span><br><span>       bool dlcx_possible = true;</span><br><span>   struct osmo_fsm_inst *fi;</span><br><span style="color: hsl(0, 100%, 40%);">-       struct gsm_mncc mncc = {</span><br><span style="color: hsl(0, 100%, 40%);">-                .msg_type = MNCC_REL_REQ,</span><br><span style="color: hsl(0, 100%, 40%);">-               .callref = mgcp_ctx->trans->callref,</span><br><span style="color: hsl(0, 100%, 40%);">-              .cause = {</span><br><span style="color: hsl(0, 100%, 40%);">-                      .location = GSM48_CAUSE_LOC_PRN_S_LU,</span><br><span style="color: hsl(0, 100%, 40%);">-                   .coding = 0, /* FIXME */</span><br><span style="color: hsl(0, 100%, 40%);">-                        .value = GSM48_CC_CAUSE_RESOURCE_UNAVAIL</span><br><span style="color: hsl(0, 100%, 40%);">-                }</span><br><span style="color: hsl(0, 100%, 40%);">-       };</span><br><span style="color: hsl(120, 100%, 40%);">+    struct gsm_mncc mncc;</span><br><span> </span><br><span>    OSMO_ASSERT(mgcp_ctx);</span><br><span>       fi = mgcp_ctx->fsm;</span><br><span>@@ -188,6 +180,16 @@</span><br><span>         * silent because we already got informed and the higher layers might</span><br><span>         * already freed their context information (trans). */</span><br><span>       if (!mgcp_ctx->free_ctx) {</span><br><span style="color: hsl(120, 100%, 40%);">+         mncc = (struct gsm_mncc) {</span><br><span style="color: hsl(120, 100%, 40%);">+                    .msg_type = MNCC_REL_REQ,</span><br><span style="color: hsl(120, 100%, 40%);">+                     .callref = mgcp_ctx->trans->callref,</span><br><span style="color: hsl(120, 100%, 40%);">+                    .cause = {</span><br><span style="color: hsl(120, 100%, 40%);">+                            .location = GSM48_CAUSE_LOC_PRN_S_LU,</span><br><span style="color: hsl(120, 100%, 40%);">+                         .coding = 0, /* FIXME */</span><br><span style="color: hsl(120, 100%, 40%);">+                              .value = GSM48_CC_CAUSE_RESOURCE_UNAVAIL</span><br><span style="color: hsl(120, 100%, 40%);">+                      }</span><br><span style="color: hsl(120, 100%, 40%);">+             };</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>                 mncc_set_cause(&mncc, GSM48_CAUSE_LOC_TRANS_NET,</span><br><span>                                GSM48_CC_CAUSE_RESOURCE_UNAVAIL);</span><br><span>             mncc_tx_to_cc(mgcp_ctx->trans->net, MNCC_REL_REQ, &mncc);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/11275">change 11275</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/11275"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: I86983eabd999c4275dcc0e4a169ef2aa1e33c747 </div>
<div style="display:none"> Gerrit-Change-Number: 11275 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>