Change in osmo-bts[master]: osmo-bts-trx: fix potential NULL pointer dereference

[fixeria]

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/25864 )


Change subject: osmo-bts-trx: fix potential NULL pointer dereference
......................................................................

osmo-bts-trx: fix potential NULL pointer dereference

Change-Id: Ic9c1f3a3fb9c151bba4f6c3e605746fc8d43f44f
Fixes: CID#240207
---
M src/osmo-bts-trx/trx_provision_fsm.c
1 file changed, 6 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/64/25864/1

diff --git a/src/osmo-bts-trx/trx_provision_fsm.c b/src/osmo-bts-trx/trx_provision_fsm.c
index 273b491..5deecd7 100644
--- a/src/osmo-bts-trx/trx_provision_fsm.c
+++ b/src/osmo-bts-trx/trx_provision_fsm.c
@@ -369,11 +369,13 @@
 
 	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);

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ic9c1f3a3fb9c151bba4f6c3e605746fc8d43f44f
Gerrit-Change-Number: 25864
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211020/55bae7e0/attachment.htm>