Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc-nat/+/27257 )
Change subject: bsc_nat_fsm: split sccp_sap_verify_called_addr out ......................................................................
Patch Set 1:
(1 comment)
File src/osmo-bsc-nat/bsc_nat_fsm.c:
https://gerrit.osmocom.org/c/osmo-bsc-nat/+/27257/comment/f5e3011f_836879be PS1, Line 74: static int sccp_sap_verify_called_addr(struct bsc_nat_ss7_inst *src, struct osmo_sccp_addr *called_addr, I'm really not sure this belongs at application level. The SCCP stack underneath knows which users have subscribed to which SSN at which address, etc. and it will only pass those messages to the user which are intended for the user.
Think of a TCP socket where you have already bound a socket and then still do recvmsg to check if the destination address is what you expect.