On Fri, May 26, 2017 at 3:42 PM, Stephan Müller <smueller@xxxxxxxxxx> wrote: > Am Donnerstag, 25. Mai 2017, 17:23:47 CEST schrieb Harsh Jain: > > Hi Harsh, > >> Initialise ctr_completion variable before use. > > Thank you very much for catching this. > > But I think the chosen function is wrong. When we have an HMAC or Hash DRBG, this completion function does not need to be initialized. > > May I ask you to check this patch? Yup, Its working. > > Ciao > Stephan > > ---8<--- > > From 1ec5f753644eb9005ad758c9bfd75032602c9727 Mon Sep 17 00:00:00 2001 > From: Stephan Mueller <smueller@xxxxxxxxxx> > Date: Fri, 26 May 2017 12:11:31 +0200 > Subject: [PATCH] crypto: drbg- Fixes panic in wait_for_completion call > > Initialise ctr_completion variable before use. > > Signed-off-by: Harsh Jain <harshjain.prof@xxxxxxxxx> > Signed-off-by: Stephan Mueller <smueller@xxxxxxxxxx> > --- > crypto/drbg.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/crypto/drbg.c b/crypto/drbg.c > index fa749f4..433c425 100644 > --- a/crypto/drbg.c > +++ b/crypto/drbg.c > @@ -1691,6 +1691,7 @@ static int drbg_init_sym_kernel(struct drbg_state *drbg) > return PTR_ERR(sk_tfm); > } > drbg->ctr_handle = sk_tfm; > + init_completion(&drbg->ctr_completion); > > req = skcipher_request_alloc(sk_tfm, GFP_KERNEL); > if (!req) { > -- > 2.9.3 > >