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

Change subject: bsc_vty: Use gsm_bts_num() in ho_or_as()
......................................................................

bsc_vty: Use gsm_bts_num() in ho_or_as()

Change-Id: I49856a417a2be3a10c3bdf2553c52064f875ef94
---
M src/osmo-bsc/bsc_vty.c
1 file changed, 1 insertion(+), 7 deletions(-)

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




diff --git a/src/osmo-bsc/bsc_vty.c b/src/osmo-bsc/bsc_vty.c
index f04c7ec..aa31505 100644
--- a/src/osmo-bsc/bsc_vty.c
+++ b/src/osmo-bsc/bsc_vty.c
@@ -833,7 +833,6 @@
 {
        struct gsm_network *net = gsmnet_from_vty(vty);
        struct gsm_subscriber_connection *conn;
-       struct gsm_bts *bts;
        struct gsm_bts *new_bts = NULL;
        unsigned int bts_nr = atoi(argv[0]);
        unsigned int trx_nr = atoi(argv[1]);
@@ -845,12 +844,7 @@
                unsigned int bts_nr_new = atoi(argv[4]);

                /* Lookup the BTS where we want to handover to */
-               llist_for_each_entry(bts, &net->bts_list, list) {
-                       if (bts->nr == bts_nr_new) {
-                               new_bts = bts;
-                               break;
-                       }
-               }
+               new_bts = gsm_bts_num(net, bts_nr_new);

                if (!new_bts) {
                        vty_out(vty, "%% Unable to trigger handover, specified 
bts #%u does not exist %s",

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

Gerrit-MessageType: merged
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I49856a417a2be3a10c3bdf2553c52064f875ef94
Gerrit-Change-Number: 38080
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>

Reply via email to