Attention is currently required from: laforge, neels.
pespin has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/37807?usp=email )
Change subject: on RUA Connect failure, respond with RUA Disconnect ......................................................................
Patch Set 2:
(1 comment)
File src/osmo-hnbgw/hnbgw_rua.c:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/37807/comment/3a994d91_d9f55bc3?us... : PS2, Line 313: tx_disconnect: I find this goto here not really useful and adding complexity to the code, since it's only used in one code path and the function above has no general "return" statement just above.
I'd say either move this entire code block directly where it belongs, or put it in a static helper function and call it in the code path where it belongs.