pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/26786 ) Change subject: iuup: Fix decoding of 1byte-length subflow size fields ...................................................................... iuup: Fix decoding of 1byte-length subflow size fields Change-Id: I78ae9e7d46d0725ddec05e004ae22ee5da738162 --- M src/gsm/iuup.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/gsm/iuup.c b/src/gsm/iuup.c index b838dc6..f526382 100644 --- a/src/gsm/iuup.c +++ b/src/gsm/iuup.c @@ -527,7 +527,7 @@ } else { uint8_t *subflow_size = ihdr_rfci->subflow_length; for (i = 0; i < ihdr->num_subflows_per_rfci; i++) { - iui->config.subflow_sizes[ihdr_rfci->rfci][i] = osmo_load16be(subflow_size); + iui->config.subflow_sizes[ihdr_rfci->rfci][i] = *subflow_size; subflow_size++; } } -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/26786 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I78ae9e7d46d0725ddec05e004ae22ee5da738162 Gerrit-Change-Number: 26786 Gerrit-PatchSet: 2 Gerrit-Owner: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged