summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-02-01 17:00:24 +0100
committerDave Kleikamp <dave.kleikamp@oracle.com>2015-02-02 15:02:34 -0600
commit648695c74811f09a8ad80d7c3be72b8169589a64 (patch)
tree1e58584f7f4cd0b428c619a8ad2c363b95b64666
parente1f1fe798d2f2eab904624125ba924d08205f960 (diff)
downloadlinux-stable-648695c74811f09a8ad80d7c3be72b8169589a64.tar.gz
linux-stable-648695c74811f09a8ad80d7c3be72b8169589a64.tar.bz2
linux-stable-648695c74811f09a8ad80d7c3be72b8169589a64.zip
jfs: Deletion of an unnecessary check before the function call "unload_nls"
The unload_nls() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
-rw-r--r--fs/jfs/super.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/jfs/super.c b/fs/jfs/super.c
index 16c3a9556634..5d30c56ae075 100644
--- a/fs/jfs/super.c
+++ b/fs/jfs/super.c
@@ -619,8 +619,7 @@ out_mount_failed:
iput(sbi->direct_inode);
sbi->direct_inode = NULL;
out_unload:
- if (sbi->nls_tab)
- unload_nls(sbi->nls_tab);
+ unload_nls(sbi->nls_tab);
out_kfree:
kfree(sbi);
return ret;