diff options
| author |   Markus Elfring <elfring@users.sourceforge.net> | 2015-02-01 17:00:24 +0100 |
|---|---|---|
| committer |   Dave Kleikamp <dave.kleikamp@oracle.com> | 2015-02-02 15:02:34 -0600 |
| commit | 648695c74811f09a8ad80d7c3be72b8169589a64 (patch) | |
| tree | 1e58584f7f4cd0b428c619a8ad2c363b95b64666 /fs/jfs | |
| parent | jfs: get rid of homegrown endianness helpers (diff) | |
| download | linux-dev-648695c74811f09a8ad80d7c3be72b8169589a64.tar.xz linux-dev-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>
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: 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; |