Change in libosmo-sccp[master]: ipa: Don't send non-SCCP data over IPA/SCCPlite

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
Sat Oct 19 12:14:00 UTC 2019


laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/15810 )

Change subject: ipa: Don't send non-SCCP data over IPA/SCCPlite
......................................................................

ipa: Don't send non-SCCP data over IPA/SCCPlite

The IPA/SCCPlite stacking is - as the name implies - constrained to
the transport of SCCP messages. We have to reject any non-SCCP payload.

Change-Id: I5e5a2879013ee8cf08aa4199b4bee498dcb61446
Fixes: OS#4235
---
M src/ipa.c
1 file changed, 8 insertions(+), 0 deletions(-)

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



diff --git a/src/ipa.c b/src/ipa.c
index f3a7a52..d013916 100644
--- a/src/ipa.c
+++ b/src/ipa.c
@@ -57,6 +57,7 @@
 int ipa_tx_xua_as(struct osmo_ss7_as *as, struct xua_msg *xua)
 {
 	struct xua_msg_part *data_ie;
+	struct m3ua_data_hdr *data_hdr;
 	struct msgb *msg;
 	unsigned int src_len;
 	const uint8_t *src;
@@ -68,6 +69,13 @@
 	data_ie = xua_msg_find_tag(xua, M3UA_IEI_PROT_DATA);
 	if (!data_ie || data_ie->len < sizeof(struct m3ua_data_hdr))
 		return -1;
+	data_hdr = (struct m3ua_data_hdr *) data_ie->dat;
+
+	if (data_hdr->si != MTP_SI_SCCP) {
+		LOGPAS(as, DLSS7, LOGL_ERROR, "Cannot transmit non-SCCP SI (%u) to IPA peer\n",
+			data_hdr->si);
+		return -1;
+	}
 
 	/* and even the data part still has the header prepended */
 	src = data_ie->dat + sizeof(struct m3ua_data_hdr);

-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/15810
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: I5e5a2879013ee8cf08aa4199b4bee498dcb61446
Gerrit-Change-Number: 15810
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at osmocom.org>
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/20191019/b1294b41/attachment.htm>


More information about the gerrit-log mailing list