table: fix LRU hash table parameters check
authorJasvinder Singh <jasvinder.singh@intel.com>
Fri, 27 Oct 2017 10:47:30 +0000 (11:47 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Tue, 7 Nov 2017 01:00:01 +0000 (02:00 +0100)
Fixes the copy paste error in lru hash table parameters check.

Coverity issue: 198433
Fixes: b5cde2cb8c81 ("table: rework variable size key lru hash table")

Signed-off-by: Jasvinder Singh <jasvinder.singh@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
lib/librte_table/rte_table_hash_lru.c

index 139b8fb..a07392f 100644 (file)
@@ -148,15 +148,14 @@ check_params_create(struct rte_table_hash_params *params)
        }
 
        /* n_keys */
-       if ((params->n_keys == 0) ||
-               (!rte_is_power_of_2(params->n_keys))) {
+       if (params->n_keys == 0) {
                RTE_LOG(ERR, TABLE, "%s: n_keys invalid value\n", __func__);
                return -EINVAL;
        }
 
        /* n_buckets */
        if ((params->n_buckets == 0) ||
-               (!rte_is_power_of_2(params->n_keys))) {
+               (!rte_is_power_of_2(params->n_buckets))) {
                RTE_LOG(ERR, TABLE, "%s: n_buckets invalid value\n", __func__);
                return -EINVAL;
        }