Change in osmo-sgsn[master]: gprs_gmm: reintroduce calling gsm48_gmm_authorize when not handling G...

lynxis lazus gerrit-no-reply at
Tue Apr 23 11:13:34 UTC 2019

lynxis lazus has submitted this change and it was merged. ( )

Change subject: gprs_gmm: reintroduce calling gsm48_gmm_authorize when not handling GMM Attach Requests

gprs_gmm: reintroduce calling gsm48_gmm_authorize when not handling GMM Attach Requests

A security command is part of multiple procedures to ensure
integrity (optional also encryption) between MS and RNC.
It should be used for all Iu connections once.
With the rewrite of the GMM Attach FSM the use of the security command
procedure was broken for all procedures e.g. Service Request except GMM
Attach Request.

Relates: OS#3920
Change-Id: I50e8e316f06ae1a6171a6b07e4e2f0761322b779
M src/gprs/gprs_gmm.c
1 file changed, 6 insertions(+), 1 deletion(-)

  Mykola Shchetinin: Looks good to me, approved
  Jenkins Builder: Verified

diff --git a/src/gprs/gprs_gmm.c b/src/gprs/gprs_gmm.c
index b0c5902..358bff9 100644
--- a/src/gprs/gprs_gmm.c
+++ b/src/gprs/gprs_gmm.c
@@ -205,7 +205,12 @@
 		/* Continue authentication here */
 		mm->iu.ue_ctx->integrity_active = 1;
-		osmo_fsm_inst_dispatch(mm->gmm_att_req.fsm, E_IU_SECURITY_CMD_COMPLETE, NULL);
+		/* FIXME: remove gmm_authorize */
+		if (mm->pending_req != GSM48_MT_GMM_ATTACH_REQ)
+			gsm48_gmm_authorize(mm);
+		else
+			osmo_fsm_inst_dispatch(mm->gmm_att_req.fsm, E_IU_SECURITY_CMD_COMPLETE, NULL);
 		LOGP(DRANAP, LOGL_NOTICE, "Unknown event received: %i\n", type);

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I50e8e316f06ae1a6171a6b07e4e2f0761322b779
Gerrit-Change-Number: 13654
Gerrit-PatchSet: 2
Gerrit-Owner: lynxis lazus <lynxis at>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Mykola Shchetinin <mykola at>
Gerrit-Reviewer: lynxis lazus <lynxis at>
Gerrit-CC: Pau Espin Pedrol <pespin at>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list