* [PATCH 07/18] btrfs: Fix a BUG_ON boundary typo
@ 2010-03-25 12:32 Miao Xie
0 siblings, 0 replies; only message in thread
From: Miao Xie @ 2010-03-25 12:32 UTC (permalink / raw)
To: Chris Mason; +Cc: Linux Btrfs
From: Zhao Lei <zhaolei@cn.fujitsu.com>
logical's value should be [em->start, em->start + em->len),
so we need to use
BUG_ON(em->start > logical || em->start + em->len <= logical);
instead of
BUG_ON(em->start > logical || em->start + em->len < logical);
Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com>
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
---
fs/btrfs/volumes.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
index 9df8e3f..844c393 100644
--- a/fs/btrfs/volumes.c
+++ b/fs/btrfs/volumes.c
@@ -2598,7 +2598,7 @@ int btrfs_num_copies(struct btrfs_mapping_tree *map_tree, u64 logical, u64 len)
read_unlock(&em_tree->lock);
BUG_ON(!em);
- BUG_ON(em->start > logical || em->start + em->len < logical);
+ BUG_ON(em->start > logical || em->start + em->len <= logical);
map = (struct map_lookup *)em->bdev;
if (map->type & (BTRFS_BLOCK_GROUP_DUP | BTRFS_BLOCK_GROUP_RAID1))
ret = map->num_stripes;
@@ -2673,7 +2673,7 @@ again:
BUG();
}
- BUG_ON(em->start > logical || em->start + em->len < logical);
+ BUG_ON(em->start > logical || em->start + em->len <= logical);
map = (struct map_lookup *)em->bdev;
offset = logical - em->start;
--
1.6.5.2
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2010-03-25 12:32 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-25 12:32 [PATCH 07/18] btrfs: Fix a BUG_ON boundary typo Miao Xie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).