Change in osmo-msc[master]: libmsc: fix potential NULL-pointer dereferences detected by GCC's LTO

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Mon Dec 2 08:44:53 UTC 2019


laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/16357 )

Change subject: libmsc: fix potential NULL-pointer dereferences detected by GCC's LTO
......................................................................

libmsc: fix potential NULL-pointer dereferences detected by GCC's LTO

Change-Id: Ib7ba8909dce64d1b8ff3a53495fe3eefc446ed8e
---
M src/libmsc/msc_ho.c
M src/libmsc/msc_vty.c
2 files changed, 9 insertions(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/libmsc/msc_ho.c b/src/libmsc/msc_ho.c
index ac7602f..aa513eb 100644
--- a/src/libmsc/msc_ho.c
+++ b/src/libmsc/msc_ho.c
@@ -67,9 +67,12 @@
 
 void msc_ho_down_required_reject(struct msc_a *msc_a, enum gsm0808_cause cause)
 {
-	struct msc_i *msc_i = msc_a_msc_i(msc_a);
+	struct msc_i *msc_i;
 	uint32_t event;
 
+	msc_i = msc_a_msc_i(msc_a);
+	OSMO_ASSERT(msc_i);
+
 	struct ran_msg ran_enc_msg = {
 		.msg_type = RAN_MSG_HANDOVER_REQUIRED_REJECT,
 		.handover_required_reject = {
diff --git a/src/libmsc/msc_vty.c b/src/libmsc/msc_vty.c
index f833585..cd82a07 100644
--- a/src/libmsc/msc_vty.c
+++ b/src/libmsc/msc_vty.c
@@ -1148,6 +1148,11 @@
 	struct gsm_sms *sms;
 
 	sms = sms_from_text(receiver, sender_msisdn, 0, str);
+	if (!sms) {
+		LOGP(DLSMS, LOGL_ERROR, "Failed to allocate SMS\n");
+		return CMD_WARNING;
+	}
+
 	sms->protocol_id = tp_pid;
 
 	/* store in database for the queue */

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

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ib7ba8909dce64d1b8ff3a53495fe3eefc446ed8e
Gerrit-Change-Number: 16357
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191202/54730ffe/attachment.htm>


More information about the gerrit-log mailing list