|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
Stephan Mueller reported to me recently a error in random number generation in
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
the ansi cprng. If several small requests are made that are less than the
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
instances block size, the remainder for loop code doesn't increment
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
rand_data_valid in the last iteration, meaning that the last bytes in the
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
rand_data buffer gets reused on the subsequent smaller-than-a-block request for
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
random data.
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
The fix is pretty easy, just re-code the for loop to make sure that
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
rand_data_valid gets incremented appropriately
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
Reported-by: Stephan Mueller <stephan.mueller@atsec.com>
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
CC: Stephan Mueller <stephan.mueller@atsec.com>
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
CC: Petr Matousek <pmatouse@redhat.com>
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
CC: Herbert Xu <herbert@gondor.apana.org.au>
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
CC: "David S. Miller" <davem@davemloft.net>
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
---
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
crypto/ansi_cprng.c | 4 ++--
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
diff --git a/crypto/ansi_cprng.c b/crypto/ansi_cprng.c
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
index c0bb377..666f196 100644
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
--- a/crypto/ansi_cprng.c
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
+++ b/crypto/ansi_cprng.c
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
@@ -230,11 +230,11 @@ remainder:
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
*/
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
if (byte_count < DEFAULT_BLK_SZ) {
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
empty_rbuf:
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
- for (; ctx->rand_data_valid < DEFAULT_BLK_SZ;
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
- ctx->rand_data_valid++) {
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
+ while (ctx->rand_data_valid < DEFAULT_BLK_SZ) {
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
*ptr = ctx->rand_data[ctx->rand_data_valid];
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
ptr++;
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
byte_count--;
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
+ ctx->rand_data_valid++;
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
if (byte_count == 0)
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
goto done;
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
}
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
--
|
|
![](https://seccdn.libravatar.org/avatar/2b3dd3a56b56c16f6e54c1904578f5f0100d27020cdfbf6479a4e47482e712c0?s=16&d=retro) |
4d7ada7 |
1.8.3.1
|