diff options
author | Daniel Walter <dwalter@google.com> | 2019-04-02 10:43:32 +0200 |
---|---|---|
committer | Richard Weinberger <richard@nod.at> | 2019-05-07 23:18:28 +0200 |
commit | 9ca55299f2ee0a2ccf41d99bfc02d5ad3118e03b (patch) | |
tree | 51c2a5a533c3561be6dbb4325ed520ae985903f0 /arch | |
parent | 91e1e547abcd88f1e23204e62e377aaa7c93a786 (diff) | |
download | linux-9ca55299f2ee0a2ccf41d99bfc02d5ad3118e03b.tar.bz2 |
um: Do not unlock mutex that is not hold.
Return error instead of trying to unlock a mutex that is not hold.
Signed-off-by: Daniel Walter <dwalter@google.com>
Reviewed-by: Anton Ivanov <anton.ivanov@cambridgegreys.com>
Acked-by: Anton Ivanov <anton.ivanov@cambridgegreys.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/um/drivers/ubd_kern.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/um/drivers/ubd_kern.c b/arch/um/drivers/ubd_kern.c index aca09be2373e..33c1cd6a12ac 100644 --- a/arch/um/drivers/ubd_kern.c +++ b/arch/um/drivers/ubd_kern.c @@ -276,14 +276,14 @@ static int ubd_setup_common(char *str, int *index_out, char **error_out) str++; if(!strcmp(str, "sync")){ global_openflags = of_sync(global_openflags); - goto out1; + return err; } err = -EINVAL; major = simple_strtoul(str, &end, 0); if((*end != '\0') || (end == str)){ *error_out = "Didn't parse major number"; - goto out1; + return err; } mutex_lock(&ubd_lock); |