Max has submitted this change and it was merged. ( https://gerrit.osmocom.org/13238 )
Change subject: Fix TA index encoder ...................................................................... Fix TA index encoder The TAI is described as { 0 | 1 < TIMING_ADVANCE_INDEX : bit (4) > } in 3GPP TS 44.018 ยง10.5.2.16.1 so it should be encoded with if-else. Change-Id: I54482790e1cf3cb13a635a99a481250576deabaf --- M src/encoding.cpp 1 file changed, 6 insertions(+), 6 deletions(-) Approvals: Jenkins Builder: Verified Pau Espin Pedrol: Looks good to me, but someone else must approve Harald Welte: Looks good to me, approved diff --git a/src/encoding.cpp b/src/encoding.cpp index ad53b0b..e21a7fc 100644 --- a/src/encoding.cpp +++ b/src/encoding.cpp @@ -117,13 +117,13 @@ { int rc; - if (tai < 0) /* No TIMING_ADVANCE_INDEX: */ + if (tai < 0) { /* No TIMING_ADVANCE_INDEX: */ SET_0(dest); - - /* TIMING_ADVANCE_INDEX: */ - SET_1(dest); - rc = bitvec_set_u64(dest, tai, 4, false); - CHECK(rc); + } else { /* TIMING_ADVANCE_INDEX: */ + SET_1(dest); + rc = bitvec_set_u64(dest, tai, 4, false); + CHECK(rc); + } return 0; } -- To view, visit https://gerrit.osmocom.org/13238 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I54482790e1cf3cb13a635a99a481250576deabaf Gerrit-Change-Number: 13238 Gerrit-PatchSet: 4 Gerrit-Owner: Max <msur...@sysmocom.de> Gerrit-Reviewer: Daniel Willmann <dwillm...@sysmocom.de> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de>