diff options
author | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-11-21 18:19:22 +0000 |
---|---|---|
committer | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-11-21 18:19:22 +0000 |
commit | 40ff1f7b41e496dc83cdac7db94990feba93bb14 (patch) | |
tree | f1637260251b287378f4adde2a63222c2cd44b38 /main/utils.c | |
parent | c0db75dd30300be9335435859f644074b0bad3c3 (diff) |
There existed about a 1 in 4 billion chance that reading from /dev/urandom
would return LONG_MIN (1 in 9 quintillion if using 64-bit longs). Since there
is no positive equivalent of LONG_MIN, the result of labs() in this case is
unpredictable. This fixes that situation.
(closes issue #11336, reported and patched by sperreault)
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@89487 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main/utils.c')
-rw-r--r-- | main/utils.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/utils.c b/main/utils.c index 8b3448f02..83ab65ac1 100644 --- a/main/utils.c +++ b/main/utils.c @@ -1140,7 +1140,7 @@ long int ast_random(void) if (dev_urandom_fd >= 0) { int read_res = read(dev_urandom_fd, &res, sizeof(res)); if (read_res > 0) - return labs(res); + return res < 0 ? ~res : res; } #endif #ifdef linux |