Re: [PATCH 5/5] crypto: atmel-sha: fix algorihtm registration

From: Herbert Xu
Date: Mon Jan 25 2016 - 02:24:20 EST


On Fri, Jan 15, 2016 at 03:49:35PM +0100, Cyrille Pitchen wrote:
>
> +static int atmel_sha_export(struct ahash_request *req, void *out)
> +{
> + const struct atmel_sha_reqctx *ctx = ahash_request_ctx(req);
> + struct atmel_sha_state *state = out;
> +
> + memcpy(state->digest, ctx->digest, SHA512_DIGEST_SIZE);
> + memcpy(state->buffer, ctx->buffer, ctx->bufcnt);
> + state->bufcnt = ctx->bufcnt;
> + state->digcnt[0] = ctx->digcnt[0];
> + state->digcnt[1] = ctx->digcnt[1];
> + return 0;
> +}

Hmm, you're assuming that out is aligned but that is not necessarily
the case. Ditto for import.

Cheers,
--
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt