aboutsummaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorGeert Uytterhoeven <geert@linux-m68k.org>2016-10-20 00:41:48 -0400
committerDave Chinner <david@fromorbit.com>2016-10-20 00:41:48 -0400
commit1be7f9be0efa4e90547f50b8188f4e70710a1173 (patch)
treef08149589bee3be8ee9ea4cc73a09ddf6a4d34c5 /fs
parent1d55a4bfd080ff4c6c96acfccfb7cdd2615ed6c2 (diff)
xfs: Fix uninitialized variable in xfs_reflink_reserve_cow_range()
with gcc 4.1.2: fs/xfs/xfs_reflink.c: In function xfs_reflink_reserve_cow_range: fs/xfs/xfs_reflink.c:327: warning: error may be used uninitialized in this function Indeed, if "count" is zero, the function will return an uninitialized error value. While "count" is unlikely to be zero, this function is called through the public iomap API. Hence fix this by preinitializing error to zero. Fixes: 2a06705cd5954030 ("xfs: create delalloc extents in CoW fork") Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/xfs/xfs_reflink.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_reflink.c b/fs/xfs/xfs_reflink.c
index 5965e9455d91..d48a7cc2fe00 100644
--- a/fs/xfs/xfs_reflink.c
+++ b/fs/xfs/xfs_reflink.c
@@ -324,7 +324,7 @@ xfs_reflink_reserve_cow_range(
324 struct xfs_mount *mp = ip->i_mount; 324 struct xfs_mount *mp = ip->i_mount;
325 xfs_fileoff_t offset_fsb, end_fsb; 325 xfs_fileoff_t offset_fsb, end_fsb;
326 bool skipped = false; 326 bool skipped = false;
327 int error; 327 int error = 0;
328 328
329 trace_xfs_reflink_reserve_cow_range(ip, offset, count); 329 trace_xfs_reflink_reserve_cow_range(ip, offset, count);
330 330