pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gbproxy/+/26037 )
Change subject: gbproxy: Correctly extract TLLI from DL/UL unitdata ...................................................................... Patch Set 1: (1 comment) https://gerrit.osmocom.org/c/osmo-gbproxy/+/26037/1/src/gb_proxy.c File src/gb_proxy.c: https://gerrit.osmocom.org/c/osmo-gbproxy/+/26037/1/src/gb_proxy.c@1028 PS1, Line 1028: tp_inner[0].lv[BSSGP_IE_TLLI].len = 4; this looks really hacky, I'd simply store stuff into *tlli and be done with it... -- To view, visit https://gerrit.osmocom.org/c/osmo-gbproxy/+/26037 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gbproxy Gerrit-Branch: master Gerrit-Change-Id: I76896f2eee588c9a286b87b69b5b58232759a6d5 Gerrit-Change-Number: 26037 Gerrit-PatchSet: 1 Gerrit-Owner: daniel <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: lynxis lazus <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Comment-Date: Fri, 29 Oct 2021 17:56:38 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
