Attention is currently required from: neels. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/29372 )
Change subject: ipaccess-config: Initiate missing IPA osmo_link ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
File src/osmo-bsc/bts_ipaccess_nanobts.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29372/comment/96d1b1fc_e9bc7eda PS1, Line 619: OSMO_ASSERT(bts->osmo_link);
would it make sense to assert this earlier, while using the link, not only when dropping it?
the interesting path here is when dropping it, since other apps may not use it. The main problem here really is ipaccess-config reimplementing all the lowest ipa parts with its own fake code while still hooking in some middle-layer parts. But rewritting that is totally out of the scope of this patchset.