diff options
author | David Sterba <dsterba@suse.cz> | 2015-04-23 17:28:30 +0200 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2015-04-24 15:42:05 +0200 |
commit | 6e0a620ba6de8c0e174ba45a0de5ba9dbf98f2c9 (patch) | |
tree | f6fca82afad30cce9a88e128591e71c7c46afec7 /cmds-check.c | |
parent | 9ea0eff31fedace7916fd2e9d6ae3eeb6a833fbc (diff) | |
download | btrfs-progs-6e0a620ba6de8c0e174ba45a0de5ba9dbf98f2c9.tar.gz |
btrfs-progs: check: fix compile warnings in del_file_extent_hole
make[1]: Nothing to be done for `all'.
cmds-check.c: In function ‘del_file_extent_hole’:
cmds-check.c:289:26: warning: ‘prev.start’ may be used uninitialized in this function
cmds-check.c:289:26: warning: ‘prev.len’ may be used uninitialized in this function
cmds-check.c:290:26: warning: ‘next.start’ may be used uninitialized in this function
cmds-check.c:290:26: warning: ‘next.len’ may be used uninitialized in this function
Reported-by: Anand Jain <Anand.Jain@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
Diffstat (limited to 'cmds-check.c')
-rw-r--r-- | cmds-check.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/cmds-check.c b/cmds-check.c index 7eb8c21..60ef48e 100644 --- a/cmds-check.c +++ b/cmds-check.c @@ -286,8 +286,10 @@ static int del_file_extent_hole(struct rb_root *holes, { struct file_extent_hole *hole; struct file_extent_hole tmp; - struct file_extent_hole prev; - struct file_extent_hole next; + u64 prev_start = 0; + u64 prev_len = 0; + u64 next_start = 0; + u64 next_len = 0; struct rb_node *node; int have_prev = 0; int have_next = 0; @@ -307,24 +309,24 @@ static int del_file_extent_hole(struct rb_root *holes, * split(s) if they exists. */ if (start > hole->start) { - prev.start = hole->start; - prev.len = start - hole->start; + prev_start = hole->start; + prev_len = start - hole->start; have_prev = 1; } if (hole->start + hole->len > start + len) { - next.start = start + len; - next.len = hole->start + hole->len - start - len; + next_start = start + len; + next_len = hole->start + hole->len - start - len; have_next = 1; } rb_erase(node, holes); free(hole); if (have_prev) { - ret = add_file_extent_hole(holes, prev.start, prev.len); + ret = add_file_extent_hole(holes, prev_start, prev_len); if (ret < 0) return ret; } if (have_next) { - ret = add_file_extent_hole(holes, next.start, next.len); + ret = add_file_extent_hole(holes, next_start, next_len); if (ret < 0) return ret; } |