diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2015-11-13 07:55:59 -0600 |
|---|---|---|
| committer | Bob Peterson <rpeterso@redhat.com> | 2015-11-16 11:56:26 -0600 |
| commit | 6fde22426be6af261816db5941744b8d3c4c7f96 (patch) | |
| tree | 9243b4492843997d973e1aa6e88651a3016bb270 /fs/gfs2 | |
| parent | acc546fd6108cb17f87f985e4235b68756d7b01f (diff) | |
| download | lwn-6fde22426be6af261816db5941744b8d3c4c7f96.tar.gz lwn-6fde22426be6af261816db5941744b8d3c4c7f96.zip | |
GFS2: Delete an unnecessary check before the function call "iput"
The iput() 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: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'fs/gfs2')
| -rw-r--r-- | fs/gfs2/ops_fstype.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/gfs2/ops_fstype.c b/fs/gfs2/ops_fstype.c index baab99b69d8a..1f9de173c4a0 100644 --- a/fs/gfs2/ops_fstype.c +++ b/fs/gfs2/ops_fstype.c @@ -910,8 +910,7 @@ fail_qc_i: fail_ut_i: iput(sdp->sd_sc_inode); fail: - if (pn) - iput(pn); + iput(pn); return error; } |