diff options
author | David Sterba <dsterba@suse.cz> | 2015-02-24 12:57:18 -0500 |
---|---|---|
committer | Chris Mason <clm@fb.com> | 2015-03-02 17:04:45 -0500 |
commit | 1932b7be973b554ffe20a5bba6ffaed6fa995cdc (patch) | |
tree | 7ac067e4c7643336d9e58335eb7b36c4761c4646 /fs | |
parent | 5cdf83edb8e41cad1ec8eab2d402b4f9d9eb7ee0 (diff) |
btrfs: fix lost return value due to variable shadowing
A block-local variable stores error code but btrfs_get_blocks_direct may
not return it in the end as there's a ret defined in the function scope.
CC: <stable@vger.kernel.org> # 3.6+
Fixes: d187663ef24c ("Btrfs: lock extents as we map them in DIO")
Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/btrfs/inode.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 8564d8ce03de..91a87f53be3c 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c | |||
@@ -7288,7 +7288,6 @@ static int btrfs_get_blocks_direct(struct inode *inode, sector_t iblock, | |||
7288 | ((BTRFS_I(inode)->flags & BTRFS_INODE_NODATACOW) && | 7288 | ((BTRFS_I(inode)->flags & BTRFS_INODE_NODATACOW) && |
7289 | em->block_start != EXTENT_MAP_HOLE)) { | 7289 | em->block_start != EXTENT_MAP_HOLE)) { |
7290 | int type; | 7290 | int type; |
7291 | int ret; | ||
7292 | u64 block_start, orig_start, orig_block_len, ram_bytes; | 7291 | u64 block_start, orig_start, orig_block_len, ram_bytes; |
7293 | 7292 | ||
7294 | if (test_bit(EXTENT_FLAG_PREALLOC, &em->flags)) | 7293 | if (test_bit(EXTENT_FLAG_PREALLOC, &em->flags)) |