Attention is currently required from: fixeria, osmith.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/34467?usp=email )
Change subject: sccp: Introduce initial support for SCCP LUDT + LUDTS messages ......................................................................
Patch Set 3:
(1 comment)
File src/sccp2sua.c:
https://gerrit.osmocom.org/c/libosmo-sccp/+/34467/comment/ed203ec3_18f55a95 PS2, Line 1449: sccp_longdata_to_sua_ptr(xua, SUA_IEI_DATA, (uint8_t *)&ludt->variable_data);
there's probably a reason, but to be sure: why not run sccp_longptr_part_consistent again here, and […]
You mean for the optional? part? because the DATA part is being checked above it. For the optional part, simply because I'm following the exact same pattern as existing UDT and XUDT, so in any case it would have to be fixed/improved in all places. I can have a look after having this patch merged, since it's the one being required right now (and proven to work).