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/.
Vadim Yanitskiy gerrit-no-reply at lists.osmocom.orgVadim Yanitskiy has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/14324 Change subject: common/rsl.c: RSL_IE_HANDO_REF is mandatory for handover CHAN ACT ...................................................................... common/rsl.c: RSL_IE_HANDO_REF is mandatory for handover CHAN ACT According to 3GPP TS 48.058, section 8.4.1, the Handover Reference element must be included if channel activation type is 'handover'. Let's properly reject CHANnel ACTivation messages with missing RSL_IE_HANDO_REF. Otherwise such requests are misinterpreted as regular (non-handover) channel requests. Found using TC_ho_rach() TTCN-3 test case. Change-Id: I9c50e1dbeb54c5470560adcdfb2bdf5abbe47993 --- M src/common/rsl.c 1 file changed, 7 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/24/14324/1 diff --git a/src/common/rsl.c b/src/common/rsl.c index b96182f..410b38a 100644 --- a/src/common/rsl.c +++ b/src/common/rsl.c @@ -1090,9 +1090,13 @@ memset(&lchan->encr, 0, sizeof(lchan->encr)); /* 9.3.9 Handover Reference */ - if ((type == RSL_ACT_INTER_ASYNC || - type == RSL_ACT_INTER_SYNC) && - TLVP_PRES_LEN(&tp, RSL_IE_HANDO_REF, 1)) { + if ((type == RSL_ACT_INTER_ASYNC || type == RSL_ACT_INTER_SYNC)) { + /* According to 8.4.1, the Handover Reference element is included + * if activation type is handover. Assuming it's mandatory. */ + if (!TLVP_PRES_LEN(&tp, RSL_IE_HANDO_REF, 1)) { + LOGPLCHAN(lchan, DRSL, LOGL_NOTICE, "Missing Handover Reference IE\n"); + return rsl_tx_chan_act_nack(lchan, RSL_ERR_MAND_IE_ERROR); + } lchan->ho.active = HANDOVER_ENABLED; lchan->ho.ref = *TLVP_VAL(&tp, RSL_IE_HANDO_REF); } -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14324 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I9c50e1dbeb54c5470560adcdfb2bdf5abbe47993 Gerrit-Change-Number: 14324 Gerrit-PatchSet: 1 Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com> Gerrit-MessageType: newchange -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190601/eb791dc4/attachment.htm>