Commit ff50dc56 authored Dec 06, 2006 by Hugh Dickins Committed by Linus Torvalds Dec 07, 2006

[PATCH] ext3 balloc: fix off-by-one against rsv_end



rsv_end is the last block within the reservation, so alloc_new_reservation
should accept start_block == rsv_end as success.

Signed-off-by: Mingming Cao <cmm@us.ibm.com>
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Cc: <linux-ext4@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

parent 16502423

fs/ext3/balloc.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1148,7 +1148,7 @@ static int alloc_new_reservation(struct ext3_reserve_window_node *my_rsv,
		* check if the first free block is within the
		* free space we just reserved
		*/
		if (start_block >= my_rsv->rsv_start && start_block < my_rsv->rsv_end)
		if (start_block >= my_rsv->rsv_start && start_block <= my_rsv->rsv_end)
		return 0; /* success */
		/*
		* if the first free bit we found is out of the reservable space