laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve pespin: Looks good to me, approved
ctrl: take both address and port from vty config

Change-Id: Iecae15333a4e77b38d9bce8c8e591af3d5fcf347
---
M include/osmocom/hlr/hlr.h
M src/ctrl.c
M src/hlr.c
3 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/include/osmocom/hlr/hlr.h b/include/osmocom/hlr/hlr.h
index e8df5cd..3acb3c5 100644
--- a/include/osmocom/hlr/hlr.h
+++ b/include/osmocom/hlr/hlr.h
@@ -48,7 +48,6 @@

/* Control Interface */
struct ctrl_handle *ctrl;
- const char *ctrl_bind_addr;

/* Local bind addr */
char *gsup_bind_addr;
diff --git a/src/ctrl.c b/src/ctrl.c
index 11b0f5c..4e4afeb 100644
--- a/src/ctrl.c
+++ b/src/ctrl.c
@@ -771,11 +771,8 @@
struct ctrl_handle *hlr_controlif_setup(struct hlr *hlr)
{
int rc;
- struct ctrl_handle *hdl = ctrl_interface_setup_dynip2(hlr,
- hlr->ctrl_bind_addr,
- OSMO_CTRL_PORT_HLR,
- hlr_ctrl_node_lookup,
- _LAST_CTRL_NODE_HLR);
+ struct ctrl_handle *hdl = ctrl_interface_setup2(hlr, OSMO_CTRL_PORT_HLR, hlr_ctrl_node_lookup,
+ _LAST_CTRL_NODE_HLR);
if (!hdl)
return NULL;

diff --git a/src/hlr.c b/src/hlr.c
index 193dd38..037ebfe 100644
--- a/src/hlr.c
+++ b/src/hlr.c
@@ -830,7 +830,6 @@
}
proxy_init(g_hlr->gs);

- g_hlr->ctrl_bind_addr = ctrl_vty_get_bind_addr();
g_hlr->ctrl = hlr_controlif_setup(g_hlr);

dgsm_start(hlr_ctx);

To view, visit change 30665. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: Iecae15333a4e77b38d9bce8c8e591af3d5fcf347
Gerrit-Change-Number: 30665
Gerrit-PatchSet: 1
Gerrit-Owner: msuraev <msuraev@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged