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;
-
pespin