diff options
author | Oliver Smith <osmith@sysmocom.de> | 2019-05-10 14:39:37 +0200 |
---|---|---|
committer | Harald Welte <laforge@gnumonks.org> | 2019-05-14 08:19:52 +0000 |
commit | ffd522ec6fcd878c8be8c06d44efb7bcb9084791 (patch) | |
tree | 546a4c333e0a2acce62521cfacf9042f9d53ae73 /src/libvlr | |
parent | c5a8e9f19a574e74304a89c8785bf762b726ae8e (diff) |
vlr_lu_fsm.c: assert for invalid events
In state machine callback functions, instead of logging an error when
an invalid event arrives, do OSMO_ASSERT(0).
Change-Id: If5363ae37b414a0ac195e5f89664c75cbad0bb21
Diffstat (limited to 'src/libvlr')
-rw-r--r-- | src/libvlr/vlr_lu_fsm.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/src/libvlr/vlr_lu_fsm.c b/src/libvlr/vlr_lu_fsm.c index 27196d844..15ab88c24 100644 --- a/src/libvlr/vlr_lu_fsm.c +++ b/src/libvlr/vlr_lu_fsm.c @@ -1086,8 +1086,7 @@ static void lu_fsm_wait_imeisv(struct osmo_fsm_inst *fi, uint32_t event, _start_lu_main(fi); break; default: - LOGPFSML(fi, LOGL_ERROR, "event without effect: %s\n", - osmo_fsm_event_name(fi->fsm, event)); + OSMO_ASSERT(0); break; } } @@ -1104,8 +1103,7 @@ static void lu_fsm_wait_pvlr(struct osmo_fsm_inst *fi, uint32_t event, vlr_loc_upd_want_imsi(fi); break; default: - LOGPFSML(fi, LOGL_ERROR, "event without effect: %s\n", - osmo_fsm_event_name(fi->fsm, event)); + OSMO_ASSERT(0); break; } } @@ -1170,8 +1168,7 @@ static void lu_fsm_wait_imsi(struct osmo_fsm_inst *fi, uint32_t event, vlr_loc_upd_node1(fi); break; default: - LOGPFSML(fi, LOGL_ERROR, "event without effect: %s\n", - osmo_fsm_event_name(fi->fsm, event)); + OSMO_ASSERT(0); break; } } @@ -1213,8 +1210,7 @@ static void lu_fsm_wait_hlr_ul_res(struct osmo_fsm_inst *fi, uint32_t event, } break; default: - LOGPFSML(fi, LOGL_ERROR, "event without effect: %s\n", - osmo_fsm_event_name(fi->fsm, event)); + OSMO_ASSERT(0); break; } } @@ -1264,8 +1260,7 @@ static void lu_fsm_wait_lu_compl(struct osmo_fsm_inst *fi, uint32_t event, lu_fsm_failure(fi, cause); break; default: - LOGPFSML(fi, LOGL_ERROR, "event without effect: %s\n", - osmo_fsm_event_name(fi->fsm, event)); + OSMO_ASSERT(0); break; } } @@ -1297,8 +1292,7 @@ static void lu_fsm_wait_lu_compl_standalone(struct osmo_fsm_inst *fi, lu_fsm_failure(fi, cause); break; default: - LOGPFSML(fi, LOGL_ERROR, "event without effect: %s\n", - osmo_fsm_event_name(fi->fsm, event)); + OSMO_ASSERT(0); break; } } |