Messages in this thread | | | Date | Thu, 20 Jan 2011 13:51:13 +0300 | From | Evgeniy Polyakov <> | Subject | Re: [PATCH] pohmelfs/dir.c: Remove unneeded mutex_unlock() from pohmelfs_rename() |
| |
On Thu, Jan 20, 2011 at 12:13:44AM +0300, Alexey Khoroshilov (khoroshilov@ispras.ru) wrote: > I do not see any reason for the mutex_unlock(&inode->i_mutex); in > pohmelfs_rename(). > > > Found by Linux Driver Verification project (linuxtesting.org).
Yes, that's a bug, we already unlocked i_mutex in common path and should not do that on error.
Thank you. Greg, please apply.
> Signed-off-by: Alexey Khoroshilov <khoroshilov@ispras.ru>
Acked-by: Evgeniy Polyakov <zbr@ioremap.net>
-- Evgeniy Polyakov
| |