diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2011-11-08 09:04:20 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2011-11-08 09:04:20 -0500 |
commit | 87654896ca619ff64f94d3881d6bd0ec7b29e25f (patch) | |
tree | d5f19dca46c1d6aaa20a3a13acfe739eec0cc203 /fs/gfs2/file.c | |
parent | dfe4d34b39b80faff52489f950a18523da7581bf (diff) |
GFS2: More automated code analysis fixes
A potentially uninitialised variable, some unreachable code,
and the main part of this, fixing the error path in the
unlink function.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/file.c')
-rw-r--r-- | fs/gfs2/file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c index 46f6f9ac1ebc..6336bc6bf458 100644 --- a/fs/gfs2/file.c +++ b/fs/gfs2/file.c | |||
@@ -609,7 +609,7 @@ static int gfs2_fsync(struct file *file, loff_t start, loff_t end, | |||
609 | struct inode *inode = mapping->host; | 609 | struct inode *inode = mapping->host; |
610 | int sync_state = inode->i_state & (I_DIRTY_SYNC|I_DIRTY_DATASYNC); | 610 | int sync_state = inode->i_state & (I_DIRTY_SYNC|I_DIRTY_DATASYNC); |
611 | struct gfs2_inode *ip = GFS2_I(inode); | 611 | struct gfs2_inode *ip = GFS2_I(inode); |
612 | int ret, ret1 = 0; | 612 | int ret = 0, ret1 = 0; |
613 | 613 | ||
614 | if (mapping->nrpages) { | 614 | if (mapping->nrpages) { |
615 | ret1 = filemap_fdatawrite_range(mapping, start, end); | 615 | ret1 = filemap_fdatawrite_range(mapping, start, end); |