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

Mark O'Donovan shiftee at posteo.net
Thu Aug 29 01:13:20 PDT 2024


Sorry, it is not a patch. It is a section of Hannes' patch.
I used the In-Reply-To / Reply-To / mailto header from the web archive
as I am not currently subscribed to the list.
I guess I should have added the > indent characters manually.

Ar Domh, 2024-08-25 ag 11:55 +0300, Sagi Grimberg scríobh:
> > 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