[MERGED] libosmo-sccp[master]: SUA: Make sure to reject unknown CO message types

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/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Mon Apr 17 08:46:18 UTC 2017


Harald Welte has submitted this change and it was merged.

Change subject: SUA: Make sure to reject unknown CO message types
......................................................................


SUA: Make sure to reject unknown CO message types

Found using sua-sgp-mtr-i-04 from Michael Tuexen's sua-testtool.

Change-Id: Iec5c8deb6cc48a1269fd85243c1350bafd1dd815
---
M src/sua.c
1 file changed, 20 insertions(+), 0 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/sua.c b/src/sua.c
index f003f7b..fb7545e 100644
--- a/src/sua.c
+++ b/src/sua.c
@@ -519,6 +519,26 @@
 {
 	struct osmo_sccp_instance *inst = asp->inst->sccp;
 
+	OSMO_ASSERT(xua->hdr.msg_class == SUA_MSGC_CO);
+
+	switch (xua->hdr.msg_type) {
+	case 0: /* Reserved, permitted by ETSI TS 101 592 5.2.3.2 */
+	case SUA_CO_CORE:
+	case SUA_CO_COAK:
+	case SUA_CO_COREF:
+	case SUA_CO_RELRE:
+	case SUA_CO_RELCO:
+	case SUA_CO_RESCO:
+	case SUA_CO_RESRE:
+	case SUA_CO_CODT:
+	case SUA_CO_CODA:
+	case SUA_CO_COERR:
+	case SUA_CO_COIT:
+		break;
+	default:
+		return SUA_ERR_UNSUPP_MSG_TYPE;
+	}
+
 	/* We feed into SCRC, which then hands the message into
 	 * either SCLC or SCOC, or forwards it to MTP */
 	return scrc_rx_mtp_xfer_ind_xua(inst, xua);

-- 
To view, visit https://gerrit.osmocom.org/2384
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iec5c8deb6cc48a1269fd85243c1350bafd1dd815
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list