Attention is currently required from: laforge.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/35123?usp=email )
Change subject: Add a GSM RLP decoder and encoder ......................................................................
Patch Set 2:
(6 comments)
File include/osmocom/gsm/rlp.h:
https://gerrit.osmocom.org/c/libosmocore/+/35123/comment/e6e943e7_0d077a3a PS2, Line 24: GSM A5 ciphering algorithm copy-paste
https://gerrit.osmocom.org/c/libosmocore/+/35123/comment/db8bb2e2_82c0f279 PS2, Line 28: 24.002 It should actually be TS 24.022.
https://gerrit.osmocom.org/c/libosmocore/+/35123/comment/4d173cf8_6ad9a36f PS2, Line 75: osmo_rlp_decode also expose `osmo_rlp_encode`?
File src/gsm/libosmogsm.map:
https://gerrit.osmocom.org/c/libosmocore/+/35123/comment/9ecd737a_db24db2c PS2, Line 863: osmo_rlp_decode also expose `osmo_rlp_encode`?
File src/gsm/rlp.c:
https://gerrit.osmocom.org/c/libosmocore/+/35123/comment/e86caeff_197bfc48 PS2, Line 77: EINVAL `ENOTSUP`
https://gerrit.osmocom.org/c/libosmocore/+/35123/comment/3c8d5198_9efab098 PS2, Line 121: EINVAL `ENOTSUP`
-
fixeria