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
This commit is contained in:
Alexander Couzens 2019-08-19 15:22:25 +02:00 committed by lynxis lazus
parent b08a380420
commit eb1b03a98a
1 changed files with 6 additions and 16 deletions

View File

@ -917,11 +917,9 @@ static int vlr_subscr_handle_lu_err(struct vlr_subscr *vsub,
return 0; return 0;
} }
static void gmm_cause_to_fsm_and_mm_cause(enum gsm48_gmm_cause gmm_cause, static void gmm_cause_to_mm_cause(enum gsm48_gmm_cause gmm_cause,
enum osmo_fsm_term_cause *fsm_cause_p, enum gsm48_reject_value *gsm48_rej_p)
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; enum gsm48_reject_value gsm48_rej = GSM48_REJECT_NETWORK_FAILURE;
switch (gmm_cause) { switch (gmm_cause) {
case GMM_CAUSE_IMSI_UNKNOWN: case GMM_CAUSE_IMSI_UNKNOWN:
@ -1004,16 +1002,8 @@ static void gmm_cause_to_fsm_and_mm_cause(enum gsm48_gmm_cause gmm_cause,
gsm48_rej = GSM48_REJECT_NETWORK_FAILURE; gsm48_rej = GSM48_REJECT_NETWORK_FAILURE;
break; break;
} }
switch (gmm_cause) {
/* refine any error causes here? */ *gsm48_rej_p = gsm48_rej;
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;
} }
/* Handle LOCATION CANCEL request from HLR */ /* Handle LOCATION CANCEL request from HLR */
@ -1021,7 +1011,7 @@ static int vlr_subscr_handle_cancel_req(struct vlr_subscr *vsub,
const struct osmo_gsup_message *gsup_msg) const struct osmo_gsup_message *gsup_msg)
{ {
enum gsm48_reject_value gsm48_rej; 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}; struct osmo_gsup_message gsup_reply = {0};
int rc, is_update_procedure = !gsup_msg->cancel_type || int rc, is_update_procedure = !gsup_msg->cancel_type ||
gsup_msg->cancel_type == OSMO_GSUP_CANCEL_TYPE_UPDATE; gsup_msg->cancel_type == OSMO_GSUP_CANCEL_TYPE_UPDATE;
@ -1033,7 +1023,7 @@ static int vlr_subscr_handle_cancel_req(struct vlr_subscr *vsub,
gsup_reply.message_type = OSMO_GSUP_MSGT_LOCATION_CANCEL_RESULT; gsup_reply.message_type = OSMO_GSUP_MSGT_LOCATION_CANCEL_RESULT;
rc = vlr_subscr_tx_gsup_message(vsub, &gsup_reply); 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_cancel_attach_fsm(vsub, fsm_cause, gsm48_rej);
vlr_subscr_rx_imsi_detach(vsub); vlr_subscr_rx_imsi_detach(vsub);