aboutsummaryrefslogtreecommitdiffstats
path: root/arch/s390/crypto/prng.c
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2016-11-18 06:11:00 -0500
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-01-12 05:39:33 -0500
commita05aa258b237535df67926204b359b1dacc03ed1 (patch)
tree0d17675c2335f6b0949676b616a9d04dc270d951 /arch/s390/crypto/prng.c
parent5d7d362abc408e69ee229459fba21f833f2f5bf1 (diff)
s390/crypto: unlock on error in prng_tdes_read()
commit 9e6e7c74315095fd40f41003850690c711e44420 upstream. We added some new locking but forgot to unlock on error. Fixes: 57127645d79d ("s390/zcrypt: Introduce new SHA-512 based Pseudo Random Generator.") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'arch/s390/crypto/prng.c')
-rw-r--r--arch/s390/crypto/prng.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/arch/s390/crypto/prng.c b/arch/s390/crypto/prng.c
index 9cc050f9536c..1113389d0a39 100644
--- a/arch/s390/crypto/prng.c
+++ b/arch/s390/crypto/prng.c
@@ -507,8 +507,10 @@ static ssize_t prng_tdes_read(struct file *file, char __user *ubuf,
507 prng_data->prngws.byte_counter += n; 507 prng_data->prngws.byte_counter += n;
508 prng_data->prngws.reseed_counter += n; 508 prng_data->prngws.reseed_counter += n;
509 509
510 if (copy_to_user(ubuf, prng_data->buf, chunk)) 510 if (copy_to_user(ubuf, prng_data->buf, chunk)) {
511 return -EFAULT; 511 ret = -EFAULT;
512 break;
513 }
512 514
513 nbytes -= chunk; 515 nbytes -= chunk;
514 ret += chunk; 516 ret += chunk;