<p>lynxis lazus <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15479">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved
  laforge: Looks good to me, but someone else must approve

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mm_gb_fsm: unassign the llme when entering MM_IDLE<br><br>MM_IDLE means there is no known state to this MM.<br><br>Change-Id: Ieb0d6cea828842763c13942fe1a63dd89399f799<br>---<br>M src/sgsn/gprs_mm_state_gb_fsm.c<br>1 file changed, 11 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/sgsn/gprs_mm_state_gb_fsm.c b/src/sgsn/gprs_mm_state_gb_fsm.c</span><br><span>index 2056540..4e1ed48 100644</span><br><span>--- a/src/sgsn/gprs_mm_state_gb_fsm.c</span><br><span>+++ b/src/sgsn/gprs_mm_state_gb_fsm.c</span><br><span>@@ -1,6 +1,7 @@</span><br><span> #include <osmocom/core/tdef.h></span><br><span> </span><br><span> #include <osmocom/sgsn/gprs_mm_state_gb_fsm.h></span><br><span style="color: hsl(120, 100%, 40%);">+#include <osmocom/sgsn/gprs_llc.h></span><br><span> </span><br><span> #include <osmocom/sgsn/debug.h></span><br><span> #include <osmocom/sgsn/sgsn.h></span><br><span>@@ -16,6 +17,15 @@</span><br><span> #define mm_state_gb_fsm_state_chg(fi, NEXT_STATE) \</span><br><span>         osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, mm_state_gb_fsm_timeouts, sgsn->cfg.T_defs, -1)</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static void st_mm_idle_on_enter(struct osmo_fsm_inst *fi, uint32_t prev_state) {</span><br><span style="color: hsl(120, 100%, 40%);">+ struct sgsn_mm_ctx *ctx = fi->priv;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      if (ctx->gb.llme) {</span><br><span style="color: hsl(120, 100%, 40%);">+                gprs_llgmm_unassign(ctx->gb.llme);</span><br><span style="color: hsl(120, 100%, 40%);">+         ctx->gb.llme = NULL;</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> static void st_mm_idle(struct osmo_fsm_inst *fi, uint32_t event, void *data)</span><br><span> {</span><br><span>  switch(event) {</span><br><span>@@ -59,6 +69,7 @@</span><br><span>  [ST_MM_IDLE] = {</span><br><span>             .in_event_mask = X(E_MM_GPRS_ATTACH) | X(E_MM_PDU_RECEPTION),</span><br><span>                .out_state_mask = X(ST_MM_READY),</span><br><span style="color: hsl(120, 100%, 40%);">+             .onenter = st_mm_idle_on_enter,</span><br><span>              .name = "Idle",</span><br><span>            .action = st_mm_idle,</span><br><span>        },</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15479">change 15479</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/c/osmo-sgsn/+/15479"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ieb0d6cea828842763c13942fe1a63dd89399f799 </div>
<div style="display:none"> Gerrit-Change-Number: 15479 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>