Attention is currently required from: osmith, laforge, pespin, fixeria. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc-nat/+/27712 )
Change subject: Add subscr_conn_fsm ......................................................................
Patch Set 6: Code-Review+2
(2 comments)
File include/osmocom/bsc_nat/subscr_conn_fsm.h:
https://gerrit.osmocom.org/c/osmo-bsc-nat/+/27712/comment/c6fc3f01_d5ed874e PS6, Line 31: (doesn't the linter complain about trailing blank lines?)
File src/osmo-bsc-nat/bssap_conn.c:
https://gerrit.osmocom.org/c/osmo-bsc-nat/+/27712/comment/04835439_e1639fc9 PS4, Line 47: uint8_t tag = tag_order[i];
I've looked at the osmo-hnbgw implementation, it's for ASN.1. […]
Pau is correct that it would indeed be possible to copy the part leading up to the changed IE and then copy the remaining part after the changed IE (or even memmove in the existing buffer), but i think it would be more code than the current patch set. i like the simplicity of the current patch set.