diff options
author | Matthew Wilcox <mawilcox@microsoft.com> | 2017-01-29 01:52:55 -0500 |
---|---|---|
committer | Matthew Wilcox <mawilcox@microsoft.com> | 2017-02-13 21:44:07 -0500 |
commit | 6da0396cac7692b6667c09382a746593fff90e6d (patch) | |
tree | ea53cfb2caa296e7aa62eeff3238f8277cb19877 /tools/testing/radix-tree/regression2.c | |
parent | 18d0c57394e42ff536e5fdc776b6b217fdd9889c (diff) |
radix tree test suite: Fix leaks in regression2.c
None of the malloc'ed data structures were ever being freed. Found with
-fsanitize=address.
Signed-off-by: Matthew Wilcox <mawilcox@microsoft.com>
Reviewed-by: Rehas Sachdeva <aquannie@gmail.com>
Diffstat (limited to 'tools/testing/radix-tree/regression2.c')
-rw-r--r-- | tools/testing/radix-tree/regression2.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tools/testing/radix-tree/regression2.c b/tools/testing/radix-tree/regression2.c index a24d1beec7c8..42dd2a33ed24 100644 --- a/tools/testing/radix-tree/regression2.c +++ b/tools/testing/radix-tree/regression2.c | |||
@@ -103,7 +103,7 @@ void regression2_test(void) | |||
103 | 103 | ||
104 | /* 4. */ | 104 | /* 4. */ |
105 | for (i = max_slots - 1; i >= 0; i--) | 105 | for (i = max_slots - 1; i >= 0; i--) |
106 | radix_tree_delete(&mt_tree, i); | 106 | free(radix_tree_delete(&mt_tree, i)); |
107 | 107 | ||
108 | /* 5. */ | 108 | /* 5. */ |
109 | // NOTE: start should not be 0 because radix_tree_gang_lookup_tag_slot | 109 | // NOTE: start should not be 0 because radix_tree_gang_lookup_tag_slot |
@@ -114,7 +114,9 @@ void regression2_test(void) | |||
114 | PAGECACHE_TAG_TOWRITE); | 114 | PAGECACHE_TAG_TOWRITE); |
115 | 115 | ||
116 | /* We remove all the remained nodes */ | 116 | /* We remove all the remained nodes */ |
117 | radix_tree_delete(&mt_tree, max_slots); | 117 | free(radix_tree_delete(&mt_tree, max_slots)); |
118 | |||
119 | BUG_ON(!radix_tree_empty(&mt_tree)); | ||
118 | 120 | ||
119 | printv(1, "regression test 2, done\n"); | 121 | printv(1, "regression test 2, done\n"); |
120 | } | 122 | } |