[PATCH 3/9] nvme: add nvme_auth_generate_digest()

Sagi Grimberg sagi at grimberg.me
Sun Aug 25 01:55:35 PDT 2024


Your patch is not formatted correctly.
Please create a proper patch and use git-send-email to send it to
the mailing list.


On 21/08/2024 13:18, Mark O'Donovan wrote:
> +	hmac_name = nvme_auth_hmac_name(hmac_id);
> +	if (!hmac_name) {
> +		pr_warn("%s: invalid hash algoritm %d\n",
> algoritm should be algorithm.
> This typo is also present in other patches.
>
>
> +	ret = base64_encode(digest, digest_len, hmac);
> +	if (ret < hmac_len)
> +		ret = -ENOKEY;
> +	*ret_digest = hmac;
> +	ret = 0;
> The value assigned to ret in the if statement will be overwritten.
> It looks like a goto is missing.
>
> Regards,
> Mark
>




More information about the Linux-nvme mailing list