pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved
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(-)

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 change 38995. To unsubscribe, or for help writing mail filters, visit settings.

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@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>