On Mon, Jun 20, 2016 at 01:25:46PM -0700, Megha Dey wrote:
>
> -     desc->tfm = child;
> -     desc->flags = CRYPTO_TFM_REQ_MAY_SLEEP;  /* check this again */
> +     ahash_request_set_tfm(desc, child);
> +     ahash_request_set_callback(desc, CRYPTO_TFM_REQ_MAY_SLEEP, NULL, NULL);

The callback function needs to be set, or this will crash when
it returns -EINPROGRESS.

>       if (err) {
>               req->base.complete = rctx->complete;

This is not calling the completion function conditinoally, rather
it is trying to restore the original completion function when we
are done.

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

Reply via email to