summaryrefslogtreecommitdiffstats
path: root/crypto
diff options
context:
space:
mode:
authorJason A. Donenfeld <Jason@zx2c4.com>2017-07-16 19:22:06 +0200
committerHerbert Xu <herbert@gondor.apana.org.au>2017-07-28 17:56:00 +0800
commitc2176f0098d994050c88d34d37c551d033f2f18f (patch)
treeb419a0bc96214cd9e896da300f380bb3ea013a3c /crypto
parent8a1012d3f2abcef43470f879dbfd72651818e059 (diff)
downloadop-kernel-dev-c2176f0098d994050c88d34d37c551d033f2f18f.zip
op-kernel-dev-c2176f0098d994050c88d34d37c551d033f2f18f.tar.gz
crypto: rng - ensure that the RNG is ready before using
Otherwise, we might be seeding the RNG using bad randomness, which is dangerous. The one use of this function from within the kernel -- not from userspace -- is being removed (keys/big_key), so that call site isn't relevant in assessing this. Cc: Herbert Xu <herbert@gondor.apana.org.au> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto')
-rw-r--r--crypto/rng.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/crypto/rng.c b/crypto/rng.c
index 5e84692..b4a6186 100644
--- a/crypto/rng.c
+++ b/crypto/rng.c
@@ -43,12 +43,14 @@ int crypto_rng_reset(struct crypto_rng *tfm, const u8 *seed, unsigned int slen)
if (!buf)
return -ENOMEM;
- get_random_bytes(buf, slen);
+ err = get_random_bytes_wait(buf, slen);
+ if (err)
+ goto out;
seed = buf;
}
err = crypto_rng_alg(tfm)->seed(tfm, seed, slen);
-
+out:
kzfree(buf);
return err;
}
OpenPOWER on IntegriCloud