laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/41495?usp=email )
Change subject: sigtran: Fix potential null ptr derreference ......................................................................
sigtran: Fix potential null ptr derreference
Related: Coverity CID#551476 Change-Id: Iab7c237dd0ab5d6e916ee0a680559cdf838c509f --- M src/osmo-bsc/osmo_bsc_sigtran.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved osmith: Looks good to me, approved
diff --git a/src/osmo-bsc/osmo_bsc_sigtran.c b/src/osmo-bsc/osmo_bsc_sigtran.c index db8b6f3..1fe7739 100644 --- a/src/osmo-bsc/osmo_bsc_sigtran.c +++ b/src/osmo-bsc/osmo_bsc_sigtran.c @@ -888,12 +888,12 @@ msc->a.mtp_user = osmo_ss7_user_find_by_si(s7i, MTP_SI_NI11_OSMO_IPA); if (!msc->a.mtp_user) { msc->a.mtp_user = osmo_ss7_user_create(s7i, "MGCP/CTRL-IPA"); + if (!msc->a.mtp_user) + return -EINVAL; osmo_ss7_user_set_prim_cb(msc->a.mtp_user, mtp_sap_up); osmo_ss7_user_set_priv(msc->a.mtp_user, s7i); osmo_ss7_user_register(msc->a.mtp_user, MTP_SI_NI11_OSMO_IPA); } - if (!msc->a.mtp_user) - return -EINVAL; }
/* Start MSC-Reset procedure */