diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-06-01 08:23:11 +0000 |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2010-06-11 11:46:12 -0400 |
| commit | 0e4dcbef1c0c3e29f9c7f824359445d385b2649a (patch) | |
| tree | cb360bec033ae7bbf237aedd84528c83f2715d47 /fs/btrfs | |
| parent | 836097797236fd727f82ec2f3f376ac41a430876 (diff) | |
| download | op-kernel-dev-0e4dcbef1c0c3e29f9c7f824359445d385b2649a.zip op-kernel-dev-0e4dcbef1c0c3e29f9c7f824359445d385b2649a.tar.gz | |
Btrfs: uninitialized data is check_path_shared()
refs can be used with uninitialized data if btrfs_lookup_extent_info()
fails on the first pass through the loop. In the original code if that
happens then check_path_shared() probably returns 1, this patch
changes it to return 1 for safety.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index d999c53..f08427c 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -2673,7 +2673,7 @@ static int check_path_shared(struct btrfs_root *root, struct extent_buffer *eb; int level; int ret; - u64 refs; + u64 refs = 1; for (level = 0; level < BTRFS_MAX_LEVEL; level++) { if (!path->nodes[level]) |
