summaryrefslogtreecommitdiffstats
path: root/bgpd/bgp_nb_config.c
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@nvidia.com>2020-11-06 02:42:03 +0100
committerDonald Sharp <sharpd@nvidia.com>2020-11-06 02:44:31 +0100
commite5209541f2f425c25ef016be2cade323b327ea17 (patch)
tree7e81b6fe7e9d7a836f9e40aba3c1e2e886a9066f /bgpd/bgp_nb_config.c
parentMerge pull request #7227 from chiragshah6/yang_dev (diff)
downloadfrr-e5209541f2f425c25ef016be2cade323b327ea17.tar.xz
frr-e5209541f2f425c25ef016be2cade323b327ea17.zip
bgpd: Actually return the group peer
The code is returning the group peer data structure, which is what is happening but we should not have assignment statements in this return statement for a `struct peer *` return. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to '')
-rw-r--r--bgpd/bgp_nb_config.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/bgpd/bgp_nb_config.c b/bgpd/bgp_nb_config.c
index 096fd11d1..a15cb7cad 100644
--- a/bgpd/bgp_nb_config.c
+++ b/bgpd/bgp_nb_config.c
@@ -6273,11 +6273,10 @@ int bgp_neighbors_unnumbered_neighbor_afi_safis_afi_safi_enabled_destroy(
static struct peer *bgp_peer_group_peer_lookup(struct bgp *bgp,
const char *peer_str)
{
- struct peer *peer = NULL;
struct peer_group *group = NULL;
group = peer_group_lookup(bgp, peer_str);
- return peer = group->conf;
+ return group->conf;
}
/*