lynxis lazus has uploaded this change for review.

View Change

gprs_gmm: refactor handling of GMM Detach Accept into own function

Change-Id: Iae846bbff0df04295a1579017117b3e84fa29157
---
M src/sgsn/gprs_gmm.c
1 file changed, 9 insertions(+), 3 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/00/43600/1
diff --git a/src/sgsn/gprs_gmm.c b/src/sgsn/gprs_gmm.c
index c35d54f..e82bc06 100644
--- a/src/sgsn/gprs_gmm.c
+++ b/src/sgsn/gprs_gmm.c
@@ -1502,6 +1502,14 @@
return rc;
}

+/* CHapter 9.4.6: MT Detach Ack */
+static int gsm48_rx_gmm_det_accept(struct sgsn_mm_ctx *ctx, struct msgb *msg)
+{
+ LOGMMCTXP(LOGL_INFO, ctx, "-> DETACH ACK\n");
+ mm_ctx_cleanup_free(ctx, "GMM DETACH ACK");
+ return 0;
+}
+
/* Chapter 9.4.15: Routing area update accept */
static int gsm48_tx_gmm_ra_upd_ack(struct sgsn_mm_ctx *mm)
{
@@ -2100,9 +2108,7 @@
case GSM48_MT_GMM_DETACH_ACK:
if (!mmctx)
goto null_mmctx;
- LOGMMCTXP(LOGL_INFO, mmctx, "-> DETACH ACK\n");
- mm_ctx_cleanup_free(mmctx, "GMM DETACH ACK");
- rc = 0;
+ rc = gsm48_rx_gmm_det_accept(mmctx, msg);
break;
case GSM48_MT_GMM_ATTACH_COMPL:
if (!mmctx)

To view, visit change 43600. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Iae846bbff0df04295a1579017117b3e84fa29157
Gerrit-Change-Number: 43600
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis@fe80.eu>