Change in libosmo-sccp[master]: vty: ss7: Make sure ASP has at least INET_ANY addr set if not configu...

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Wed Oct 23 17:30:52 UTC 2019


pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/15868 )


Change subject: vty: ss7: Make sure ASP has at least INET_ANY addr set if not configured in VTY
......................................................................

vty: ss7: Make sure ASP has at least INET_ANY addr set if not configured in VTY

Several parts of the code expect an operative ASP to always have at
least one local ip addr, be it INET_ANY (NULL, 0.0.0.0) or any specific
IP address.
This allows osmo_ss7_asp_find_by_socket_addr() matching incoming
connectiongs against ASPs defined through VTY "asp" node with no
"local-ip" cmd option specified. In that case, we assume the user wants
it to bind/match against INET_ANY.

Related: OS#3608
Fixes: 80b135581909fef595d48436ab04dbcb147e3895
Change-Id: Id4da0c778890c90cf5bfcf0fd3a7ecf165f2a6df
---
M src/osmo_ss7_vty.c
1 file changed, 5 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/68/15868/1

diff --git a/src/osmo_ss7_vty.c b/src/osmo_ss7_vty.c
index 8169aaf..803d6cd 100644
--- a/src/osmo_ss7_vty.c
+++ b/src/osmo_ss7_vty.c
@@ -1722,6 +1722,11 @@
 	switch (vty->node) {
 	case L_CS7_ASP_NODE:
 		asp = vty->index;
+		/* If no local addr was set */
+		if (!asp->cfg.local.host_cnt) {
+			asp->cfg.local.host[0] = NULL;
+			asp->cfg.local.host_cnt = 1;
+		}
 		osmo_ss7_asp_restart(asp);
 		vty->node = L_CS7_NODE;
 		vty->index = asp->inst;

-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/15868
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: Id4da0c778890c90cf5bfcf0fd3a7ecf165f2a6df
Gerrit-Change-Number: 15868
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191023/af2dae54/attachment.htm>


More information about the gerrit-log mailing list