Mesa (main): util/hash_table: Use FREE() to be consistent with the CALLOC_STRUCT() call

[GitLab Mirror]

Module: Mesa
Branch: main
Commit: 977cc3157dc0215c065bc3039f7fd5c8b359dc17
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=977cc3157dc0215c065bc3039f7fd5c8b359dc17

Author: Boris Brezillon <boris.brezil...@collabora.com>
Date:   Fri Dec  1 18:39:31 2023 +0100

util/hash_table: Use FREE() to be consistent with the CALLOC_STRUCT() call

CALLOC_STRUCT() calls the OS abstraction layer to do the allocation.
Call FREE() to free the corresponding objects so we keep things
consistent and have proper debug traces when memory-debugging
is enabled.

Fixes: 6649b840c340 ("mesa/util: add a hash table wrapper which support 64-bit 
keys")
Signed-off-by: Boris Brezillon <boris.brezil...@collabora.com>
Reviewed-by: Yonggang Luo <luoyongg...@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26423>

---

 src/util/hash_table.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/util/hash_table.c b/src/util/hash_table.c
index dc00b2de8e9..a832d589309 100644
--- a/src/util/hash_table.c
+++ b/src/util/hash_table.c
@@ -802,7 +802,7 @@ _mesa_hash_table_u64_delete_key(struct hash_entry *entry)
    struct hash_key_u64 *_key = (struct hash_key_u64 *)entry->key;
 
    if (_key)
-      free(_key);
+      FREE(_key);
 }
 
 void
@@ -905,6 +905,6 @@ _mesa_hash_table_u64_remove(struct hash_table_u64 *ht, 
uint64_t key)
       struct hash_key *_key = (struct hash_key *)entry->key;
 
       _mesa_hash_table_remove(ht->table, entry);
-      free(_key);
+      FREE(_key);
    }
 }