diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-02-01 11:00:24 -0500 |
---|---|---|
committer | Dave Kleikamp <dave.kleikamp@oracle.com> | 2015-02-02 16:02:34 -0500 |
commit | 648695c74811f09a8ad80d7c3be72b8169589a64 (patch) | |
tree | 1e58584f7f4cd0b428c619a8ad2c363b95b64666 /fs/jfs | |
parent | e1f1fe798d2f2eab904624125ba924d08205f960 (diff) |
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>
Diffstat (limited to 'fs/jfs')
-rw-r--r-- | fs/jfs/super.c | 3 |
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: | |||
619 | iput(sbi->direct_inode); | 619 | iput(sbi->direct_inode); |
620 | sbi->direct_inode = NULL; | 620 | sbi->direct_inode = NULL; |
621 | out_unload: | 621 | out_unload: |
622 | if (sbi->nls_tab) | 622 | unload_nls(sbi->nls_tab); |
623 | unload_nls(sbi->nls_tab); | ||
624 | out_kfree: | 623 | out_kfree: |
625 | kfree(sbi); | 624 | kfree(sbi); |
626 | return ret; | 625 | return ret; |