* [patch] hfsplus: fix double lock typo in ioctl
@ 2010-10-25 4:53 Dan Carpenter
2010-10-25 10:55 ` Arnd Bergmann
2010-10-25 18:44 ` Christoph Hellwig
0 siblings, 2 replies; 3+ messages in thread
From: Dan Carpenter @ 2010-10-25 4:53 UTC (permalink / raw)
To: Christoph Hellwig
Cc: Frederic Weisbecker, Arnd Bergmann, linux-kernel, kernel-janitors
This was supposed to be a mutex_unlock() instead of a mutex_lock().
Signed-off-by: Dan Carpenter <error27@gmail.com>
diff --git a/fs/hfsplus/ioctl.c b/fs/hfsplus/ioctl.c
index 5b4667e..40a85a3 100644
--- a/fs/hfsplus/ioctl.c
+++ b/fs/hfsplus/ioctl.c
@@ -92,7 +92,7 @@ static int hfsplus_ioctl_setflags(struct file *file, int __user *user_flags)
mark_inode_dirty(inode);
out_unlock_inode:
- mutex_lock(&inode->i_mutex);
+ mutex_unlock(&inode->i_mutex);
out_drop_write:
mnt_drop_write(file->f_path.mnt);
out:
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [patch] hfsplus: fix double lock typo in ioctl
2010-10-25 4:53 [patch] hfsplus: fix double lock typo in ioctl Dan Carpenter
@ 2010-10-25 10:55 ` Arnd Bergmann
2010-10-25 18:44 ` Christoph Hellwig
1 sibling, 0 replies; 3+ messages in thread
From: Arnd Bergmann @ 2010-10-25 10:55 UTC (permalink / raw)
To: Dan Carpenter
Cc: Christoph Hellwig, Frederic Weisbecker, linux-kernel,
kernel-janitors
On Monday 25 October 2010, Dan Carpenter wrote:
> This was supposed to be a mutex_unlock() instead of a mutex_lock().
>
> Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [patch] hfsplus: fix double lock typo in ioctl
2010-10-25 4:53 [patch] hfsplus: fix double lock typo in ioctl Dan Carpenter
2010-10-25 10:55 ` Arnd Bergmann
@ 2010-10-25 18:44 ` Christoph Hellwig
1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2010-10-25 18:44 UTC (permalink / raw)
To: Dan Carpenter, Frederic Weisbecker, Arnd Bergmann, linux-kernel,
kernel-janitors
On Mon, Oct 25, 2010 at 06:53:27AM +0200, Dan Carpenter wrote:
> This was supposed to be a mutex_unlock() instead of a mutex_lock().
>
> Signed-off-by: Dan Carpenter <error27@gmail.com>
Thanks,
applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-10-25 18:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-25 4:53 [patch] hfsplus: fix double lock typo in ioctl Dan Carpenter
2010-10-25 10:55 ` Arnd Bergmann
2010-10-25 18:44 ` Christoph Hellwig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox