| Message ID | 1488589.sO3PXAYh7F@tachyon.chronox.de (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Herbert Xu |
| Headers | show |
On Fri, Mar 13, 2015 at 11:44:07AM +0100, Stephan Mueller wrote: > Due to the change to RNGs to always return zero in success case, the RNG > interface must zeroize the buffer with the length provided by the > caller. > > Signed-off-by: Stephan Mueller <smueller@chronox.de> Applied.
diff --git a/crypto/algif_rng.c b/crypto/algif_rng.c index 67f612c..a346173 100644 --- a/crypto/algif_rng.c +++ b/crypto/algif_rng.c @@ -87,7 +87,7 @@ static int rng_recvmsg(struct kiocb *unused, struct socket *sock, return genlen; err = memcpy_to_msg(msg, result, len); - memzero_explicit(result, genlen); + memzero_explicit(result, len); return err ? err : len; }
Due to the change to RNGs to always return zero in success case, the RNG interface must zeroize the buffer with the length provided by the caller. Signed-off-by: Stephan Mueller <smueller@chronox.de> --- crypto/algif_rng.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)