Change in ...osmo-msc[master]: vlr: gmm_cause_to_fsm_and_mm_cause() drop fsm_cause_p argument

[lynxis lazus]

lynxis lazus has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/osmo-msc/+/15490 )

Change subject: vlr: gmm_cause_to_fsm_and_mm_cause() drop fsm_cause_p argument
......................................................................

vlr: gmm_cause_to_fsm_and_mm_cause() drop fsm_cause_p argument

It's always set to OSMO_TERM_ERROR. Move the assignment to the caller.
In prepartion to use gmm_cause_to_fsm_and_mm_cause() in vlr_auth_fsm.

Change-Id: Ie4720ad40ef7bcfc528d8d63bfc606c9c0545fb2
---
M src/libvlr/vlr.c
1 file changed, 6 insertions(+), 16 deletions(-)

Approvals:
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c
index ea160b1..a227240 100644
--- a/src/libvlr/vlr.c
+++ b/src/libvlr/vlr.c
@@ -917,11 +917,9 @@
 	return 0;
 }
 
-static void gmm_cause_to_fsm_and_mm_cause(enum gsm48_gmm_cause gmm_cause,
-					  enum osmo_fsm_term_cause *fsm_cause_p,
-					  enum gsm48_reject_value *gsm48_rej_p)
+static void gmm_cause_to_mm_cause(enum gsm48_gmm_cause gmm_cause,
+				  enum gsm48_reject_value *gsm48_rej_p)
 {
-	enum osmo_fsm_term_cause fsm_cause = OSMO_FSM_TERM_ERROR;
 	enum gsm48_reject_value gsm48_rej = GSM48_REJECT_NETWORK_FAILURE;
 	switch (gmm_cause) {
 	case GMM_CAUSE_IMSI_UNKNOWN:
@@ -1004,16 +1002,8 @@
 		gsm48_rej = GSM48_REJECT_NETWORK_FAILURE;
 		break;
 	}
-	switch (gmm_cause) {
-		/* refine any error causes here? */
-	default:
-		fsm_cause = OSMO_FSM_TERM_ERROR;
-		break;
-	}
-	if (fsm_cause_p)
-		*fsm_cause_p = fsm_cause;
-	if (gsm48_rej_p)
-		*gsm48_rej_p = gsm48_rej;
+
+	*gsm48_rej_p = gsm48_rej;
 }
 
 /* Handle LOCATION CANCEL request from HLR */
@@ -1021,7 +1011,7 @@
 					const struct osmo_gsup_message *gsup_msg)
 {
 	enum gsm48_reject_value gsm48_rej;
-	enum osmo_fsm_term_cause fsm_cause;
+	enum osmo_fsm_term_cause fsm_cause = OSMO_FSM_TERM_ERROR;
 	struct osmo_gsup_message gsup_reply = {0};
 	int rc, is_update_procedure = !gsup_msg->cancel_type ||
 		gsup_msg->cancel_type == OSMO_GSUP_CANCEL_TYPE_UPDATE;
@@ -1033,7 +1023,7 @@
 	gsup_reply.message_type = OSMO_GSUP_MSGT_LOCATION_CANCEL_RESULT;
 	rc = vlr_subscr_tx_gsup_message(vsub, &gsup_reply);
 
-	gmm_cause_to_fsm_and_mm_cause(gsup_msg->cause, &fsm_cause, &gsm48_rej);
+	gmm_cause_to_mm_cause(gsup_msg->cause, &gsm48_rej);
 	vlr_subscr_cancel_attach_fsm(vsub, fsm_cause, gsm48_rej);
 
 	vlr_subscr_rx_imsi_detach(vsub);

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/15490
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ie4720ad40ef7bcfc528d8d63bfc606c9c0545fb2
Gerrit-Change-Number: 15490
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190916/32d8b0b9/attachment.htm>