> > - 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