> 
> -     if (!key || key_len != ((I40E_PFQF_HKEY_MAX_INDEX + 1) *
> -             sizeof(uint32_t)))
> +     if (!key || key_len == 0) {
> +             PMD_DRV_LOG(DEBUG, "No key to be configured");
> +             return 0;
> +     } else if (key && (key_len != (I40E_PFQF_HKEY_MAX_INDEX + 1) *
> +             sizeof(uint32_t))) {
No need to check key in else if, it shouldn't be NULL here.
> +             PMD_DRV_LOG(ERR, "Invalid key length %u", key_len);
>               return -EINVAL;
> +     }

Thanks
Jingjing

Reply via email to