Re: [patch] crypto: cleanup: remove unneeded null check

2010-05-02 Thread Herbert Xu
On Wed, Apr 28, 2010 at 10:55:15AM +0200, Dan Carpenter wrote: > We don't check "frontend" consistently in crypto_init_spawn2(). We > check it at the start of the function but then we dereference it > unconditionally in the parameter list when we call crypto_init_spawn(). > > I looked at the pla

[patch] crypto: cleanup: remove unneeded null check

2010-04-28 Thread Dan Carpenter
We don't check "frontend" consistently in crypto_init_spawn2(). We check it at the start of the function but then we dereference it unconditionally in the parameter list when we call crypto_init_spawn(). I looked at the places that call crypto_init_spawn2() and "frontend" is always a valid point