diff options
author | Stephen Worley <sworley@cumulusnetworks.com> | 2019-10-04 19:54:39 +0200 |
---|---|---|
committer | Stephen Worley <sworley@cumulusnetworks.com> | 2019-10-25 17:13:43 +0200 |
commit | cd36b87d8c067626dce566143e7998d0a0a9b722 (patch) | |
tree | 8c9897ae084522d31343aca30160977d763038c5 /pbrd | |
parent | zebra: Use ng pointer in mpls_ftn_uninstall (diff) | |
download | frr-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.c | 1 |
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; |