random: Fix handing of arch_get_random_long in get_random_bytes()
authorLuck, Tony <tony.luck@intel.com>
Wed, 16 Nov 2011 18:50:56 +0000 (10:50 -0800)
committerH. Peter Anvin <hpa@linux.intel.com>
Thu, 17 Nov 2011 00:41:02 +0000 (16:41 -0800)
If there is an architecture-specific random number generator we use
it to acquire randomness one "long" at a time. We should put these
random words into consecutive words in the result buffer - not just
overwrite the first word again and again.

Signed-off-by: Tony Luck <tony.luck@intel.com>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/4ec4061010261a4cb0@agluck-desktop.sc.intel.com
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
drivers/char/random.c

index 63e19ba..6035ab8 100644 (file)
@@ -941,7 +941,7 @@ void get_random_bytes(void *buf, int nbytes)
                if (!arch_get_random_long(&v))
                        break;
                
                if (!arch_get_random_long(&v))
                        break;
                
-               memcpy(buf, &v, chunk);
+               memcpy(p, &v, chunk);
                p += chunk;
                nbytes -= chunk;
        }
                p += chunk;
                nbytes -= chunk;
        }