diff options
author | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2016-10-13 14:43:49 +0200 |
---|---|---|
committer | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2016-10-13 15:17:06 +0200 |
commit | c7ccdd490f84914754c72c7b3105772f74a40208 (patch) | |
tree | 0f3e535e9a655ed4c0da787452f8a628df091de8 /src | |
parent | 4ff1a5a957859ceec50f3f191399da5163198ff5 (diff) |
cosmetic: hnbgw: addr related renames, move define, move comment
Prepare for parameterization of IuCS and IuPS addresses:
Conform internal variable naming to local-ip, local-port, remote-ip,
remote-port (instead of bind-ip).
Rename HNBGW_IUH_LOCAL_IP_DEFAULT to HNGGW_LOCAL_IP_DEFAULT to be more general
and move it to the top.
Move a function doc comment to the .c file.
Change-Id: Ice85941c978498e3ddf41d151248507e7f56cb5d
Diffstat (limited to 'src')
-rw-r--r-- | src/hnbgw.c | 20 | ||||
-rw-r--r-- | src/hnbgw_vty.c | 10 |
2 files changed, 17 insertions, 13 deletions
diff --git a/src/hnbgw.c b/src/hnbgw.c index 8d6dbb1..eb36367 100644 --- a/src/hnbgw.c +++ b/src/hnbgw.c @@ -76,7 +76,7 @@ static struct hnb_gw *hnb_gw_create(void *ctx) { struct hnb_gw *gw = talloc_zero(ctx, struct hnb_gw); - gw->config.iuh_listen_port = IUH_DEFAULT_SCTP_PORT; + gw->config.iuh_local_port = IUH_DEFAULT_SCTP_PORT; gw->next_ue_ctx_id = 23; INIT_LLIST_HEAD(&gw->hnb_list); @@ -308,11 +308,15 @@ static int accept_cb(struct osmo_stream_srv_link *srv, int fd) return 0; } -const char *hnbgw_get_iuh_bind_addr(struct hnb_gw *gw) +/* + * Return IP address passed to the hnbgw/iuh/local-ip command, or + * HNBGW_LOCAL_IP_DEFAULT. + */ +const char *hnbgw_get_iuh_local_ip(struct hnb_gw *gw) { - const char *addr = gw->config.iuh_bind_addr; + const char *addr = gw->config.iuh_local_ip; if (!addr) - addr = HNBGW_IUH_LOCAL_IP_DEFAULT; + addr = HNBGW_LOCAL_IP_DEFAULT; return addr; } @@ -507,8 +511,8 @@ int main(int argc, char **argv) g_hnb_gw->cnlink_ps = hnbgw_cnlink_init(g_hnb_gw, "127.0.0.2", SUA_PORT, 1); LOGP(DMAIN, LOGL_NOTICE, "Listening for Iuh at %s %d\n", - hnbgw_get_iuh_bind_addr(g_hnb_gw), - g_hnb_gw->config.iuh_listen_port); + hnbgw_get_iuh_local_ip(g_hnb_gw), + g_hnb_gw->config.iuh_local_port); srv = osmo_stream_srv_link_create(tall_hnb_ctx); if (!srv) { perror("cannot create server"); @@ -516,8 +520,8 @@ int main(int argc, char **argv) } osmo_stream_srv_link_set_data(srv, g_hnb_gw); osmo_stream_srv_link_set_proto(srv, IPPROTO_SCTP); - osmo_stream_srv_link_set_addr(srv, hnbgw_get_iuh_bind_addr(g_hnb_gw)); - osmo_stream_srv_link_set_port(srv, g_hnb_gw->config.iuh_listen_port); + osmo_stream_srv_link_set_addr(srv, hnbgw_get_iuh_local_ip(g_hnb_gw)); + osmo_stream_srv_link_set_port(srv, g_hnb_gw->config.iuh_local_port); osmo_stream_srv_link_set_accept_cb(srv, accept_cb); if (osmo_stream_srv_link_open(srv) < 0) { diff --git a/src/hnbgw_vty.c b/src/hnbgw_vty.c index 5dee41a..93c1f9a 100644 --- a/src/hnbgw_vty.c +++ b/src/hnbgw_vty.c @@ -107,10 +107,10 @@ DEFUN(show_talloc, show_talloc_cmd, "show talloc", SHOW_STR "Display talloc info DEFUN(cfg_hnbgw_iuh_local_ip, cfg_hnbgw_iuh_local_ip_cmd, "local-ip A.B.C.D", "Accept Iuh connections on local interface\n" - "Local interface IP address (default: " HNBGW_IUH_LOCAL_IP_DEFAULT ")") + "Local interface IP address (default: " HNBGW_LOCAL_IP_DEFAULT ")") { - talloc_free((void*)g_hnb_gw->config.iuh_bind_addr); - g_hnb_gw->config.iuh_bind_addr = talloc_strdup(tall_hnb_ctx, argv[0]); + talloc_free((void*)g_hnb_gw->config.iuh_local_ip); + g_hnb_gw->config.iuh_local_ip = talloc_strdup(tall_hnb_ctx, argv[0]); return CMD_SUCCESS; } @@ -136,8 +136,8 @@ static int config_write_hnbgw_iuh(struct vty *vty) vty_out(vty, " iuh%s", VTY_NEWLINE); - addr = g_hnb_gw->config.iuh_bind_addr; - if (addr && (strcmp(addr, HNBGW_IUH_LOCAL_IP_DEFAULT) != 0)) + addr = g_hnb_gw->config.iuh_local_ip; + if (addr && (strcmp(addr, HNBGW_LOCAL_IP_DEFAULT) != 0)) vty_out(vty, " local-ip %s%s", addr, VTY_NEWLINE); if (g_hnb_gw->config.hnbap_allow_tmsi) |