hwrng: core - Fix double unlock in rng_dev_read
authorHerbert Xu <herbert@gondor.apana.org.au>
Wed, 23 Dec 2009 15:22:34 +0000 (23:22 +0800)
committerHerbert Xu <herbert@gondor.apana.org.au>
Wed, 23 Dec 2009 15:22:34 +0000 (23:22 +0800)
When the loop terminates with size == 0 in rng_dev_read we will
unlock the rng mutex twice.

Reported-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/char/hw_random/core.c

index e989f67..3d9c61e 100644 (file)
@@ -158,10 +158,11 @@ static ssize_t rng_dev_read(struct file *filp, char __user *buf,
                        goto out;
                }
        }
-out_unlock:
-       mutex_unlock(&rng_mutex);
 out:
        return ret ? : err;
+out_unlock:
+       mutex_unlock(&rng_mutex);
+       goto out;
 }