laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/34867?usp=email )

 (

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted 
one.
 )Change subject: ASCI: Correctly set and store 'uplink-release-local' flag
......................................................................

ASCI: Correctly set and store 'uplink-release-local' flag

Also this option is enabled by default and show in the VTY if it is
disabled.

Change-Id: I6af44f0dd7ff842de633587fb0dcbe78126d30e1
Related: OS#5364
---
M src/host/layer23/src/common/settings.c
M src/host/layer23/src/mobile/vty_interface.c
2 files changed, 16 insertions(+), 1 deletion(-)

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




diff --git a/src/host/layer23/src/common/settings.c 
b/src/host/layer23/src/common/settings.c
index 395d239..e79a6a1 100644
--- a/src/host/layer23/src/common/settings.c
+++ b/src/host/layer23/src/common/settings.c
@@ -116,6 +116,8 @@

        INIT_LLIST_HEAD(&set->abbrev);

+       set->uplink_release_local = true;
+
        return 0;
 }

diff --git a/src/host/layer23/src/mobile/vty_interface.c 
b/src/host/layer23/src/mobile/vty_interface.c
index 7611c0f..aaf01bd 100644
--- a/src/host/layer23/src/mobile/vty_interface.c
+++ b/src/host/layer23/src/mobile/vty_interface.c
@@ -1326,7 +1326,7 @@
                        set->any_timeout, VTY_NEWLINE);
        if (!l23_vty_hide_default || !set->uplink_release_local)
                vty_out(vty, " %suplink-release-local%s",
-                       (set->uplink_release_local) ? "no " : "", VTY_NEWLINE);
+                       (!set->uplink_release_local) ? "no " : "", VTY_NEWLINE);
        if (!l23_vty_hide_default || set->asci_allow_any)
                vty_out(vty, " %sasci-allow-any%s",
                        (set->asci_allow_any) ? "" : "no ", VTY_NEWLINE);

--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34867?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I6af44f0dd7ff842de633587fb0dcbe78126d30e1
Gerrit-Change-Number: 34867
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andr...@eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-MessageType: merged

Reply via email to