fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30290 ) Change subject: trxcon: trx_data_rx_cb(): check TRXD PDU version ...................................................................... trxcon: trx_data_rx_cb(): check TRXD PDU version Change-Id: If5687d7d87619c0da02cc81ab26d63e61484bc06 --- M src/host/trxcon/src/trx_if.c 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/src/host/trxcon/src/trx_if.c b/src/host/trxcon/src/trx_if.c index c8e7a93..c938fc0 100644 --- a/src/host/trxcon/src/trx_if.c +++ b/src/host/trxcon/src/trx_if.c @@ -648,6 +648,12 @@ return -EINVAL; } + if ((buf[0] >> 4) != 0) { + LOGPFSMSL(trx->fi, DTRXD, LOGL_ERROR, + "Got TRXD PDU with unexpected version\n"); + return -ENOTSUP; + } + burst = (sbit_t *)&buf[8]; bi = (struct trxcon_phyif_burst_ind) { -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30290 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: If5687d7d87619c0da02cc81ab26d63e61484bc06 Gerrit-Change-Number: 30290 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged