Commit ee0ddadd authored Jun 10, 2008 by Heiko Carstens Committed by Martin Schwidefsky Jun 10, 2008

[S390] vmemmap: fix off-by-one bug.



If a memory range is supposed to be added to the 1:1 mapping and it
ends just below the maximum supported physical address it won't
succeed. This is because a test doesn't consider that the end address
is 1 smaller than start + size.
Fix the comparison.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

parent 12829126

arch/s390/mm/vmem.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -236,7 +236,7 @@ static int insert_memory_segment(struct memory_segment *seg)
		{
		struct memory_segment *tmp;

		if (seg->start + seg->size >= VMEM_MAX_PHYS \|\|
		if (seg->start + seg->size > VMEM_MAX_PHYS \|\|
		seg->start + seg->size < seg->start)
		return -ERANGE;