[PATCH 8/9] jfs: Add missing mutex_unlock call to error path

[email protected] Wed, 17 Jun 2009 08:44:30 -0500 (CDT)
Newsgroups gmane.comp.file-systems.jfs.patches
Message-ID <[email protected]>
commit 79f52b77b89e8b7aa9fbe62135eea198a2ecbd5b
Author: Dave Kleikamp <[email protected]>
Date:   Sat May 23 20:28:41 2009 -0500

    jfs: Add missing mutex_unlock call to error path
    
    Jan Kucera found an missing call to mutex_unlock() with his static code
    checker.  It's an unlikely error path to hit in the real world, but it
    should be fixed.
    
    Signed-off-by: Dave Kleikamp <[email protected]>
    Reported-by: Jan Kucera <[email protected]>

diff --git a/fs/jfs/jfs_imap.c b/fs/jfs/jfs_imap.c
index 3460572..0fc3040 100644
--- a/fs/jfs/jfs_imap.c
+++ b/fs/jfs/jfs_imap.c
@@ -2571,6 +2571,7 @@ diNewIAG(struct inomap * imap, int *iagnop, int agno, struct metapage ** mpp)
 
 			txAbort(tid, 0);
 			txEnd(tid);
+			mutex_unlock(&JFS_IP(ipimap)->commit_mutex);
 
 			/* release the inode map lock */
 			IWRITE_UNLOCK(ipimap);

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects