summaryrefslogtreecommitdiffstats
path: root/pbrd
diff options
context:
space:
mode:
authorStephen Worley <sworley@cumulusnetworks.com>2019-10-04 19:54:39 +0200
committerStephen Worley <sworley@cumulusnetworks.com>2019-10-25 17:13:43 +0200
commitcd36b87d8c067626dce566143e7998d0a0a9b722 (patch)
tree8c9897ae084522d31343aca30160977d763038c5 /pbrd
parentzebra: Use ng pointer in mpls_ftn_uninstall (diff)
downloadfrr-cd36b87d8c067626dce566143e7998d0a0a9b722.tar.xz
frr-cd36b87d8c067626dce566143e7998d0a0a9b722.zip
pbrd: nexthop_group delete cb don't free pbr->nhg
The pbr->nhg callback is used exclusively for individual nexthops set through `set nexthop`. If an actuall "tracked" nexthop_group is used, only the `pbrms->nhgrp_name` is set. Thus this delete does nothing. Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>
Diffstat (limited to '')
-rw-r--r--pbrd/pbr_nht.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/pbrd/pbr_nht.c b/pbrd/pbr_nht.c
index 67a1fe2ff..062cfd015 100644
--- a/pbrd/pbr_nht.c
+++ b/pbrd/pbr_nht.c
@@ -639,7 +639,6 @@ void pbr_nht_delete_group(const char *name)
if (pbrms->nhgrp_name
&& strmatch(pbrms->nhgrp_name, name)) {
pbrms->reason |= PBR_MAP_INVALID_NO_NEXTHOPS;
- nexthop_group_delete(&pbrms->nhg);
pbrms->nhg = NULL;
pbrms->internal_nhg_name = NULL;
pbrm->valid = false;