diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-04-04 06:56:30 +1100 |
|---|---|---|
| committer | Dave Chinner <david@fromorbit.com> | 2014-04-04 06:56:30 +1100 |
| commit | 805eeb8e04706a16cb0b23fd4c4abbb0bc7df82d (patch) | |
| tree | c889470e61deb167ab869b712e6cb785a6dbad38 /fs/xfs | |
| parent | fe4c224aa1ffa4352849ac5f452de7132739bee2 (diff) | |
xfs: extra semi-colon breaks a condition
There were some extra semi-colons here which mean that we return true
unintentionally.
Fixes: a49935f200e2 ('xfs: xfs_check_page_type buffer checks need help')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs')
| -rw-r--r-- | fs/xfs/xfs_aops.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/xfs/xfs_aops.c b/fs/xfs/xfs_aops.c index 5935cce8c26c..dba48853eb10 100644 --- a/fs/xfs/xfs_aops.c +++ b/fs/xfs/xfs_aops.c @@ -659,10 +659,10 @@ xfs_check_page_type( if (type == XFS_IO_UNWRITTEN) return true; } else if (buffer_delay(bh)) { - if (type == XFS_IO_DELALLOC); + if (type == XFS_IO_DELALLOC) return true; } else if (buffer_dirty(bh) && buffer_mapped(bh)) { - if (type == XFS_IO_OVERWRITE); + if (type == XFS_IO_OVERWRITE) return true; } |