Fix over-freeing in internal object cache

When returning item to the cache and the items allocated exceeded cache->limit, we should free the item and not put it to the cache.

The current code will free any item that tried returned to the cache when cache->total < cache->limit which is true in most cases if the total objects has never exceeded the limit.

1 files changed, 1 insertions, 1 deletions

diff --git a/cache.c b/cache.c
index 18dc8b7..95548e7 100644
--- a/cache.c
+++ b/cache.c
@@ -142,7 +142,7 @@ void do_cache_free(cache_t *cache, void *ptr) {
     }
     ptr = pre;
 #endif
-    if (cache->limit > cache->total) {
+    if (cache->limit != 0 && cache->limit < cache->total) {
         /* Allow freeing in case the limit was revised downward */
         if (cache->destructor) {
             cache->destructor(ptr, NULL);