diff options
author | Vadim Yanitskiy <axilirator@gmail.com> | 2019-05-14 21:49:47 +0700 |
---|---|---|
committer | Vadim Yanitskiy <axilirator@gmail.com> | 2019-05-14 21:49:47 +0700 |
commit | d24c46a38b663c4c7f19dd5c5da82a525e8ce4be (patch) | |
tree | bf5d0da392194952c438cb4208025ee911e29329 | |
parent | ede95d18b35cc89b36bebdcdf6b1222a0fa32c31 (diff) |
libmsc/ran_peer.c: avoid unreasonable use of goto in ran_peer_down_paging()
Change-Id: I3320240d8f1dc318e516162bb32e01ddafc7e30e
-rw-r--r-- | src/libmsc/ran_peer.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/libmsc/ran_peer.c b/src/libmsc/ran_peer.c index a77cb89a1..40e26a2e9 100644 --- a/src/libmsc/ran_peer.c +++ b/src/libmsc/ran_peer.c @@ -640,15 +640,11 @@ int ran_peer_down_paging(struct ran_peer *rp, const struct gsm0808_cell_id *page { struct msgb *l2; - if (cell_id_list_find(&rp->cells_seen, page_id, 0, false)) - goto page_it; - /* There are also the RAN peers that are configured in the neighbor ident for Handover, but if those aren't * connected, then we can't Page there. */ + if (!cell_id_list_find(&rp->cells_seen, page_id, 0, false)) + return 0; - return 0; - -page_it: LOG_RAN_PEER_CAT(rp, DPAG, LOGL_DEBUG, "Paging for %s on %s\n", vlr_subscr_name(vsub), gsm0808_cell_id_name(page_id)); l2 = rp->sri->ran->sccp_ran_ops.make_paging_msg(rp->sri, page_id, vsub->imsi, vsub->tmsi, cause); |