diff options
| author | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2011-04-27 09:34:33 +0200 |
|---|---|---|
| committer | Martin Schwidefsky <sky@mschwide.boeblingen.de.ibm.com> | 2011-04-27 09:34:45 +0200 |
| commit | ed961581a7ca91d6a4852af2e44333e983100505 (patch) | |
| tree | 8d3b2cf0b718879b802fc28f4eac9f3020fc3d1c /arch/s390/crypto | |
| parent | 8e10cd74342c7f5ce259cceca36f6eba084f5d58 (diff) | |
| download | linux-ed961581a7ca91d6a4852af2e44333e983100505.tar.gz linux-ed961581a7ca91d6a4852af2e44333e983100505.tar.bz2 linux-ed961581a7ca91d6a4852af2e44333e983100505.zip | |
[S390] prng: fix pointer arithmetic
The git commit c708c57e247775928b9a6bce7b4d8d14883bf39b fixed the
access beyond the end of the stack in prng_seed but the pointer
arithmetic is still incorrect. The calculation has been off by
a factor of 64, now it is only off by a factor of 8. prng_seed
is called with a maximum of 16 for nbytes, small enough that
the incorrect calculation stays insides the limits of the stack.
Place parentheses for correct pointer arithmetic.
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'arch/s390/crypto')
| -rw-r--r-- | arch/s390/crypto/prng.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/s390/crypto/prng.c b/arch/s390/crypto/prng.c index 44bca3f994b0..8b16c479585b 100644 --- a/arch/s390/crypto/prng.c +++ b/arch/s390/crypto/prng.c @@ -76,7 +76,7 @@ static void prng_seed(int nbytes) /* Add the entropy */ while (nbytes >= 8) { - *((__u64 *)parm_block) ^= *((__u64 *)buf+i); + *((__u64 *)parm_block) ^= *((__u64 *)(buf+i)); prng_add_entropy(); i += 8; nbytes -= 8; |