summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDenil Vira <denil@cumulusnetworks.com>2015-07-23 06:52:16 -0700
committerPaul Jakma <paul@quagga.net>2015-09-24 15:26:44 +0100
commit4fdb5f401eb277fa54d80e99d241bd9b03895a6a (patch)
tree9048bb26214861b9bef418c7d5bee5f213bf5d6f
parenta9d4cb33faa6af622240190a80f41c4672374925 (diff)
downloadquagga-4fdb5f401eb277fa54d80e99d241bd9b03895a6a.tar.bz2
quagga-4fdb5f401eb277fa54d80e99d241bd9b03895a6a.tar.xz
lib: Fix Free Pointer dereference in lib/filter.c
Coverity Scan ID 23056. access is accessed after free in access_list_delete Signed-off-by: Denil Vira <denil@cumulusnetworks.com>
-rw-r--r--lib/filter.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/filter.c b/lib/filter.c
index 96605c7d..a4729414 100644
--- a/lib/filter.c
+++ b/lib/filter.c
@@ -493,13 +493,13 @@ access_list_filter_delete (struct access_list *access, struct filter *filter)
filter_free (filter);
- /* If access_list becomes empty delete it from access_master. */
- if (access_list_empty (access))
- access_list_delete (access);
-
/* Run hook function. */
if (master->delete_hook)
(*master->delete_hook) (access);
+
+ /* If access_list becomes empty delete it from access_master. */
+ if (access_list_empty (access))
+ access_list_delete (access);
}
/*