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:12:02 -0700 |
| commit | 4f850fdfbb90c57178e24e9e6c9fc32d18cc23e1 (patch) | |
| tree | 2a7cb848cee55439eb0befe2b44c0f942df73708 /fs | |
| parent | 6858c106c6a0eaa458423b223154b0baf4450e03 (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 36cca1dcec8e..94eede4c93eb 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -1431,7 +1431,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; } |