From f74fa33bcee6bc84f41442bdd256d838c2cb3c14 Mon Sep 17 00:00:00 2001 From: "Dr. Stephen Henson" Date: Sun, 6 Apr 2014 23:11:20 +0100 Subject: [PATCH] Return if ssleay_rand_add called with zero num. Treat a zero length passed to ssleay_rand_add a no op: the existing logic zeroes the md value which is very bad. OpenSSL itself never does this internally and the actual call doesn't make sense as it would be passing zero bytes of entropy. Thanks to Marcus Meissner for reporting this bug. (cherry picked from commit 5be1ae28ef3c4bdec95b94f14e0e939157be550a) --- crypto/rand/md_rand.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/crypto/rand/md_rand.c b/crypto/rand/md_rand.c index 6cab3087bb..67ac5ac927 100644 --- a/crypto/rand/md_rand.c +++ b/crypto/rand/md_rand.c @@ -212,6 +212,9 @@ static int ssleay_rand_add(const void *buf, int num, double add) int do_not_lock; int rv = 0; + if (!num) + return; + /* * (Based on the rand(3) manpage) * -- 2.50.1