RE: [PATCH] crypto: talitos: fix driver init

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I was notified by a fellow member of the OpenWrt list that the talitos was already fixed in the crypto-dev repo so I backported that patch into our tree.

He also pointed out (gracefully) that I was a bit naive with regard to the fact that just setting the statesize is not enough.

Sorry for the noise.

Thanks
Alex

________________________________________
From: Kim Phillips [kim.phillips@xxxxxxx]
Sent: Saturday, April 30, 2016 1:55 AM
To: Alexandru Ardelean
Cc: linux-crypto@xxxxxxxxxxxxxxx; Alexandru Ardelean
Subject: Re: [PATCH] crypto: talitos: fix driver init

On Thu, 28 Apr 2016 17:15:30 +0300
Alexandru Ardelean <ardeleanalex@xxxxxxxxx> wrote:

> From: Alexandru Ardelean <ardeleanalex@xxxxxxxxx>
>
> Crypto hash algorithms must provide the statesize sometime
> from kernel 4.2 onwards.
> Since commit 8996eafdcbad149ac0f772fb1649fbb75c482a6a
>
> Signed-off-by: Alexandru Ardelean <aardelean@xxxxxxxxxxxx>
> ---

This should already have been fixed here:

www.spinics.net/lists/linux-crypto/msg19225.html

> @@ -2458,6 +2458,7 @@ static struct talitos_alg_template driver_algs[] = {
>       {       .type = CRYPTO_ALG_TYPE_AHASH,
>               .alg.hash = {
>                       .halg.digestsize = MD5_DIGEST_SIZE,
> +                     .halg.statesize  = sizeof(struct talitos_ahash_req_ctx),

although I'm not sure why these statesize assignments aren't being
done in talitos_alg_alloc() there either

Thanks,

Kim
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux