diff options
| author | Johannes Thumshirn <jthumshirn@suse.de> | 2018-11-28 11:54:54 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2018-12-17 16:51:41 +0300 |
| commit | cc2c39d605d23e7f1fbc376bf011fd06e46ee79f (patch) | |
| tree | c2eed4b8f7dd2b59cfb2d58b2edc2516c7ef2260 | |
| parent | a5fb11429167ee6ddeeacc554efaf5776b36433a (diff) | |
| download | linux-cc2c39d605d23e7f1fbc376bf011fd06e46ee79f.tar.xz | |
btrfs: don't initialize 'offset' in map_private_extent_buffer()
In map_private_extent_buffer() the 'offset' variable is initialized to a
page aligned version of the 'start' parameter.
But later on it is overwritten with either the offset from the extent
buffer's start or 0.
So get rid of the initial initialization.
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/extent_io.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index 582b4b1c41e0..7aafdec49dc3 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c @@ -5380,7 +5380,7 @@ int map_private_extent_buffer(const struct extent_buffer *eb, char **map, unsigned long *map_start, unsigned long *map_len) { - size_t offset = start & (PAGE_SIZE - 1); + size_t offset; char *kaddr; struct page *p; size_t start_offset = eb->start & ((u64)PAGE_SIZE - 1); |