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:14:03 -0700 |
| commit | 6d2a63f3d1bbdf0bcd9201e2826a727261c0becc (patch) | |
| tree | e345296751c2c55b9a8cbb5dea97dd60808d724a /fs | |
| parent | efb57277b9df5bcdf99aabd8b4bafe9c4cbd7576 (diff) | |
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')
| -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 1e0439111d3..997a009f3df 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -1424,7 +1424,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; } |