diff options
author | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2021-10-20 23:59:13 +0300 |
---|---|---|
committer | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2021-10-23 01:18:27 +0300 |
commit | 5f3d01eed48ba973a4901e65fc893ac43fd68347 (patch) | |
tree | a1530dbdd5be5473be3a77ca3d9d9a92ee07f1c6 | |
parent | df6e4533dd81887ca6f21792c72db1676b0971ad (diff) |
osmo-bts-trx: fix potential NULL pointer dereference
Change-Id: Ic9c1f3a3fb9c151bba4f6c3e605746fc8d43f44f
Fixes: CID#240207
-rw-r--r-- | src/osmo-bts-trx/trx_provision_fsm.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/osmo-bts-trx/trx_provision_fsm.c b/src/osmo-bts-trx/trx_provision_fsm.c index 273b4913..5deecd78 100644 --- a/src/osmo-bts-trx/trx_provision_fsm.c +++ b/src/osmo-bts-trx/trx_provision_fsm.c @@ -369,11 +369,13 @@ static void st_open_poweroff_on_enter(struct osmo_fsm_inst *fi, uint32_t prev_st trx_provision_reset(l1h); - /* Apply initial RFMUTE state */ - if (pinst->trx != NULL) - trx_if_cmd_rfmute(l1h, pinst->trx->mo.nm_state.administrative != NM_STATE_UNLOCKED); - else + if (pinst->trx == NULL) { trx_if_cmd_rfmute(l1h, true); + return; + } + + /* Apply initial RFMUTE state */ + trx_if_cmd_rfmute(l1h, pinst->trx->mo.nm_state.administrative != NM_STATE_UNLOCKED); osmo_fsm_inst_dispatch(pinst->trx->mo.fi, NM_EV_SW_ACT, NULL); osmo_fsm_inst_dispatch(pinst->trx->bb_transc.mo.fi, NM_EV_SW_ACT, NULL); |