Attention is currently required from: neels.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/33158 )
Change subject: coverity: fix type of local var ......................................................................
Patch Set 1:
(1 comment)
File src/osmo-hnbgw/context_map.c:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/33158/comment/d447288f_394edfed PS1, Line 99: if (new_scu_conn_id < 0) { iiuc, eg. now 0xffffaaaa (valid address) may be handled as a negative and fail here.
IMHO it's much better keep using uint32_t and to explicitly look for 0xFFFFFFFF for failure.