plugins/cache: Fixed a bug with destroying FIFO metadata
authorMahmoud Mandour <ma.mandourr@gmail.com>
Tue, 20 Jul 2021 23:26:55 +0000 (00:26 +0100)
committerAlex Bennée <alex.bennee@linaro.org>
Fri, 23 Jul 2021 16:22:16 +0000 (17:22 +0100)
This manifests itself when associativity degree is greater than the
number of sets and FIFO is used, otherwise it's also a memory leak
whenever FIFO was used.

Signed-off-by: Mahmoud Mandour <ma.mandourr@gmail.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210714172151.8494-2-ma.mandourr@gmail.com>
Message-Id: <20210720232703.10650-22-alex.bennee@linaro.org>

contrib/plugins/cache.c

index bf0d2f609733d74175a93bfa9915c303b856e87b..4a71602639eef82476fa7b23419ff514893d5012 100644 (file)
@@ -200,7 +200,7 @@ static void fifo_destroy(Cache *cache)
 {
     int i;
 
-    for (i = 0; i < cache->assoc; i++) {
+    for (i = 0; i < cache->num_sets; i++) {
         g_queue_free(cache->sets[i].fifo_queue);
     }
 }