pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-cbc/+/28731 )

Change subject: cbc_main: Fix setting default SBc-AP local address
......................................................................

cbc_main: Fix setting default SBc-AP local address

The issue was not spotted until now because until recently
cbc_vty_go_parent was not called due to another bug.

Change-Id: I697597438d8c45f5956b12e2a6ea76413e945e5e
---
M src/cbc_main.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved



diff --git a/src/cbc_main.c b/src/cbc_main.c
index a670722..cd80e31 100644
--- a/src/cbc_main.c
+++ b/src/cbc_main.c
@@ -88,7 +88,7 @@
        switch (vty->node) {
        case SBcAP_NODE:
                /* If no local addr set, add a default one: */
-               if (g_cbc->config.sbcap.num_local_host) {
+               if (g_cbc->config.sbcap.num_local_host == 0) {
                        g_cbc->config.sbcap.local_host[0] = 
talloc_strdup(g_cbc, "127.0.0.1");
                                g_cbc->config.sbcap.num_local_host = 1;
                }

--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28731
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I697597438d8c45f5956b12e2a6ea76413e945e5e
Gerrit-Change-Number: 28731
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged

Reply via email to