diff options
author | Josh Triplett <josht@us.ibm.com> | 2006-10-04 02:17:12 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-04 07:55:31 -0700 |
commit | 75cfef32f26d03f5d0a0833572d52f94ad858a36 (patch) | |
tree | 2440ee021888c4d743cf403a75d859060127479b /include/net/dsfield.h | |
parent | 2860aaba4dc87fa43c08724434b87a8650f3bff5 (diff) |
[PATCH] rcu: Fix sign bug making rcu_random always return the same sequence
rcu_random uses a counter rrs_count to occasionally mix data from
get_random_bytes into the state of its pseudorandom generator. However,
the rrs_counter gets declared as an unsigned long, and rcu_random checks
for --rrs_count < 0, so this code will never mix any real random data into
the state, and will thus always return the same sequence of random numbers.
Also, change the return value of rcu_random from long to unsigned long, to
avoid potential issues caused by the use of the % operator, which can
return negative values for negative left operands.
Signed-off-by: Josh Triplett <josh@freedesktop.org>
Acked-by: Paul E. McKenney <paulmck@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'include/net/dsfield.h')
0 files changed, 0 insertions, 0 deletions