On Tue, Aug 25, 2009 at 10:14:16AM +0200, Steffen Klassert wrote: > Return the value we got from crypto_register_alg() instead of > returning 0 in any case. > > Signed-off-by: Steffen Klassert <steffen.klassert@xxxxxxxxxxx> > --- > crypto/ansi_cprng.c | 9 +-------- > 1 files changed, 1 insertions(+), 8 deletions(-) > > diff --git a/crypto/ansi_cprng.c b/crypto/ansi_cprng.c > index 5357ba7..3aa6e38 100644 > --- a/crypto/ansi_cprng.c > +++ b/crypto/ansi_cprng.c > @@ -408,17 +408,10 @@ static struct crypto_alg rng_alg = { > /* Module initalization */ > static int __init prng_mod_init(void) > { > - int ret = 0; > - > if (fips_enabled) > rng_alg.cra_priority += 200; > > - ret = crypto_register_alg(&rng_alg); > - > - if (ret) > - goto out; > -out: > - return 0; > + return crypto_register_alg(&rng_alg); > } > > static void __exit prng_mod_fini(void) > -- > 1.5.4.2 > > Thanks! Acked-by: Neil Horman <nhorman@xxxxxxxxxxxxx> -- 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