Change in ...osmo-bts[master]: common/rsl.c: fix possible NULL-pointer dereference

[fixeria]

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


Change subject: common/rsl.c: fix possible NULL-pointer dereference
......................................................................

common/rsl.c: fix possible NULL-pointer dereference

Change-Id: I11a35a8f500fafa7b3c93d2f2244cc4d42f09f1b
Fixes: CID#203810
---
M src/common/rsl.c
1 file changed, 5 insertions(+), 1 deletion(-)



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

diff --git a/src/common/rsl.c b/src/common/rsl.c
index 0bcad4c..c9a5869 100644
--- a/src/common/rsl.c
+++ b/src/common/rsl.c
@@ -1740,10 +1740,14 @@
 /* 8.5.8 CBCH Load Information */
 int rsl_tx_cbch_load_indication(struct gsm_bts *bts, bool ext_cbch, bool overflow, uint8_t amount)
 {
-	struct gsm_lchan *lchan = gsm_bts_get_cbch(bts);
+	struct gsm_lchan *lchan;
 	struct msgb *msg;
 	uint8_t load_info;
 
+	lchan = gsm_bts_get_cbch(bts);
+	if (!lchan)
+		return -ENODEV;
+
 	msg = rsl_msgb_alloc(sizeof(struct abis_rsl_cchan_hdr));
 	if (!msg)
 		return -ENOMEM;

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I11a35a8f500fafa7b3c93d2f2244cc4d42f09f1b
Gerrit-Change-Number: 15439
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190906/98f646b2/attachment.htm>