diff options
| author | Igor Ryzhov <iryzhov@nfware.com> | 2021-04-14 13:08:18 +0300 |
|---|---|---|
| committer | Igor Ryzhov <iryzhov@nfware.com> | 2021-04-14 13:08:18 +0300 |
| commit | 8c4796a22d196d37a94ed497eaf5dbcd1f3a93de (patch) | |
| tree | a31c3f6bb67c0e14cb571979f5778c58ae4b74e2 /lib/filter.c | |
| parent | d75213d26036a2880f23f5e67cb1c890f20299de (diff) | |
lib: fix access-list deletion
Problems with the current implementation:
* Delete hook is called before the deletion of the access-list from the
master list, which means that daemons processing this hook successfully
find this access-list, store a pointer to it in their structures, and
right after that the access-list is freed. Daemons end up having stale
pointer to the freed structure.
* Route-maps are not notified of the deletion.
This commit fixes both issues.
Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
Diffstat (limited to 'lib/filter.c')
| -rw-r--r-- | lib/filter.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/filter.c b/lib/filter.c index ab62e95fb6..ba8a3086af 100644 --- a/lib/filter.c +++ b/lib/filter.c @@ -181,6 +181,11 @@ void access_list_delete(struct access_list *access) else list->head = access->next; + route_map_notify_dependencies(access->name, RMAP_EVENT_FILTER_DELETED); + + if (master->delete_hook) + master->delete_hook(access); + XFREE(MTYPE_ACCESS_LIST_STR, access->name); XFREE(MTYPE_TMP, access->remark); |