All of lore.kernel.org
 help / color / mirror / Atom feed
* [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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.