diff options
| author | Liu Bo <bo.li.liu@oracle.com> | 2014-06-08 19:04:13 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-06-30 20:09:46 -0700 |
| commit | d6f5d5fd0c1de1a443e3a9c3d439dd5d760f11cc (patch) | |
| tree | 694bc64fce322d6c74af2d1e795173582957a9ca /fs/btrfs | |
| parent | 2346e1e345bbb141271d0e54d6d906118db8864d (diff) | |
| download | lwn-d6f5d5fd0c1de1a443e3a9c3d439dd5d760f11cc.tar.gz lwn-d6f5d5fd0c1de1a443e3a9c3d439dd5d760f11cc.zip | |
Btrfs: use right type to get real comparison
commit cd857dd6bc2ae9ecea14e75a34e8a8fdc158e307 upstream.
We want to make sure the point is still within the extent item, not to verify
the memory it's pointing to.
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: Chris Mason <clm@fb.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/backref.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c index 290e347b6db3..75386cb004af 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -1366,7 +1366,7 @@ static int __get_extent_inline_ref(unsigned long *ptr, struct extent_buffer *eb, *out_eiref = (struct btrfs_extent_inline_ref *)(ei + 1); } *ptr = (unsigned long)*out_eiref; - if ((void *)*ptr >= (void *)ei + item_size) + if ((unsigned long)(*ptr) >= (unsigned long)ei + item_size) return -ENOENT; } |