pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-abis/+/38995?usp=email )
Change subject: ipaccess: Assert explicitly about non-null ptr
......................................................................
ipaccess: Assert explicitly about non-null ptr
Make coverity happy. Those pointers can never be NULL due to related
logic of e1i_ts_has_pending_tx_msgs() and e1inp_tx_ts().
Closes: Coverity CID#435269
Change-Id: Ifc79899bba787f08cc83206b0a53b04b66861c7b
---
M src/input/ipaccess.c
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
osmith: Looks good to me, approved
diff --git a/src/input/ipaccess.c b/src/input/ipaccess.c
index 94f1b2b..5a24686 100644
--- a/src/input/ipaccess.c
+++ b/src/input/ipaccess.c
@@ -557,6 +557,8 @@
struct e1inp_sign_link *sign_link = NULL;
struct msgb *msg;
msg = e1inp_tx_ts(e1i_ts, &sign_link);
+ OSMO_ASSERT(msg);
+ OSMO_ASSERT(sign_link);
rc |= ipaccess_bts_send_msg(e1i_ts, sign_link, cli, msg);
}
return rc;
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-abis/+/38995?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: Ifc79899bba787f08cc83206b0a53b04b66861c7b
Gerrit-Change-Number: 38995
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>