diff options
| author | Steven Whitehouse <swhiteho@redhat.com> | 2011-11-08 14:04:20 +0000 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2011-11-08 14:04:20 +0000 |
| commit | 87654896ca619ff64f94d3881d6bd0ec7b29e25f (patch) | |
| tree | d5f19dca46c1d6aaa20a3a13acfe739eec0cc203 /fs/gfs2/file.c | |
| parent | dfe4d34b39b80faff52489f950a18523da7581bf (diff) | |
| download | op-kernel-dev-87654896ca619ff64f94d3881d6bd0ec7b29e25f.zip op-kernel-dev-87654896ca619ff64f94d3881d6bd0ec7b29e25f.tar.gz | |
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 46f6f9a..6336bc6 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, struct inode *inode = mapping->host; int sync_state = inode->i_state & (I_DIRTY_SYNC|I_DIRTY_DATASYNC); struct gfs2_inode *ip = GFS2_I(inode); - int ret, ret1 = 0; + int ret = 0, ret1 = 0; if (mapping->nrpages) { ret1 = filemap_fdatawrite_range(mapping, start, end); |
