util/hash: fix removal

When removal is going to empty the bucket list, this does not mean
we don't need to compare the data.

Note: Nothing like fixing a 16 years old bug.

Ticket: 7475

src/util-hash.c

@@ -148,11 +148,14 @@ int HashTableRemove(HashTable *ht, void *data, uint16_t datalen)
     }
 
     if (ht->array[hash]->next == NULL) {
-        if (ht->Free != NULL)
-            ht->Free(ht->array[hash]->data);
-        SCFree(ht->array[hash]);
-        ht->array[hash] = NULL;
-        return 0;
+        if (ht->Compare(ht->array[hash]->data, ht->array[hash]->size, data, datalen) == 1) {
+            if (ht->Free != NULL)
+                ht->Free(ht->array[hash]->data);
+            SCFree(ht->array[hash]);
+            ht->array[hash] = NULL;
+            return 0;
+        }
+        return -1;
     }
 
     HashTableBucket *hashbucket = ht->array[hash], *prev_hashbucket = NULL;