return 0 on count error, not negative.

because the return value is unsigned.

diff --git a/src/modules/rlm_cache/drivers/rlm_cache_rbtree/rlm_cache_rbtree.c b/src/modules/rlm_cache/drivers/rlm_cache_rbtree/rlm_cache_rbtree.c
index caf90333def69c7af3b6b3e89fc4a87ac8a5edcd..07e4ca795bb112a09287410b93ef7d6863d57c5f 100644 (file)
--- a/src/modules/rlm_cache/drivers/rlm_cache_rbtree/rlm_cache_rbtree.c
+++ b/src/modules/rlm_cache/drivers/rlm_cache_rbtree/rlm_cache_rbtree.c
@@ -239,7 +239,7 @@ static uint64_t cache_entry_count(UNUSED rlm_cache_config_t const *config, void
 {
        rlm_cache_rbtree_t *driver = talloc_get_type_abort(instance, rlm_cache_rbtree_t);
 
-       if (!request) return CACHE_ERROR;
+       if (!request) return 0; /* can't return an error due to signed   */
 
        return fr_rb_num_elements(driver->mutable->cache);
 }