Attention is currently required from: pespin.

fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/libosmocore/+/43232?usp=email )

Change subject: gsm/ipa: fix t_len truncation in ipa_ccm_id_resp_parse() ......................................................................

Patch Set 1: Code-Review+2

(1 comment)

File src/gsm/ipa.c:

https://gerrit.osmocom.org/c/libosmocore/+/43232/comment/2ba28f45_c9e70f94?u... : PS1, Line 212: uint16_t t_len;

Waoh so ipa_ccm_id_resp_parse() has a 2 byte len and ipa_ccm_id_get_parse() has a 1 byte len?

Yes. See also the payload format in doxygen comments above. For this function it's "16bit length value (length of payload *and tag*)".