linux-btrfs.vger.kernel.org archive mirror
 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 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).