From 5897cfe18e3fcead96255a1a62ef706b67adcf2c Mon Sep 17 00:00:00 2001
From: Tim Duesterhus <tim@bastelstu.be>
Date: Mon, 18 Jan 2021 13:41:18 +0100
Subject: [PATCH] CLEANUP: cache: Use proper data types in secondary_key_cmp()

- hash_length is `unsigned int` and so should offset.
- idx is compared to a `size_t` and thus it should also be.
---
 src/cache.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/cache.c b/src/cache.c
index 16dc7d61c..f556fdc07 100644
--- a/src/cache.c
+++ b/src/cache.c
@@ -208,8 +208,8 @@ struct cache_entry *entry_exist(struct cache *cache, char *hash)
 static int secondary_key_cmp(const char *ref_key, const char *new_key)
 {
 	int retval = 0;
-	int idx = 0;
-	int offset = 0;
+	size_t idx = 0;
+	unsigned int offset = 0;
 	const struct vary_hashing_information *info;
 
 	for (idx = 0; idx < sizeof(vary_information)/sizeof(*vary_information) && !retval; ++idx) {