linux-kernel - [PATCH] arch:arm:mm:Aligning the module end and Correction in

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-id: <1447747011-26930-1-git-send-email-shailendra.v@samung.com>
Date:	Tue, 17 Nov 2015 13:26:51 +0530
From:	Shailendra Verma <shailendra.v@...ung.com>
To:	Russell King <linux@....linux.org.uk>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Arve Hjønnevåg <arve@...roid.com>,
	Riley Andrews <riandrews@...roid.com>,
	Laura Abbott <lauraa@...eaurora.org>,
	Jungseung Lee <js07.lee@...il.com>,
	Shailendra Verma <shailendra.capricorn@...il.com>,
	Dan Carpenter <dan.carpenter@...cle.com>,
	Mitchel Humpherys <mitchelh@...eaurora.org>,
	Sumit Semwal <sumit.semwal@...aro.org>,
	"Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
	Gioh Kim <gioh.kim@....com>,
	Rasmus Villemoes <linux@...musvillemoes.dk>,
	Shawn Lin <shawn.lin@...k-chips.com>,
	Markus Elfring <elfring@...rs.sourceforge.net>,
	linux-arm-kernel@...ts.infradead.org, devel@...verdev.osuosl.org,
	Hillf Danton <hillf.zj@...baba-inc.com>
Cc:	Ravikant Bijendra Sharma <ravikant.s2@...sung.com>,
	Shailendra Verma <shailendra.v@...sung.com>,
	linux-kernel@...r.kernel.org, vidushi.koul@...sung.com
Subject: [PATCH] arch:arm:mm:Aligning the module end and Correction in

From: Shailendra Verma <Shailendra.v@...sung.com>

The module end was not aligned as of module start and boundary
check for module end is not proper.This out of bound value of
module end can produce undesired results.

Reported-by: Hillf Danton <hillf.zj@...baba-inc.com>
Signed-off-by: Shailendra Verma <Shailendra.v@...sung.com>
Reviewed-by: Ravikant Bijendra Sharma <ravikant.s2@...sung.com>
---
 linux-4.3-rc6/arch/arm/mm/pageattr.c |   11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/linux-4.3-rc6/arch/arm/mm/pageattr.c b/linux-4.3-rc6/arch/arm/mm/pageattr.c
index cf30daf..17f103c 100644
--- a/linux-4.3-rc6/arch/arm/mm/pageattr.c
+++ b/linux-4.3-rc6/arch/arm/mm/pageattr.c
@@ -43,16 +43,15 @@ static int change_memory_common(unsigned long addr, int numpages,
 	int ret;
 	struct page_change_data data;
 
-	if (!IS_ALIGNED(addr, PAGE_SIZE)) {
-		start &= PAGE_MASK;
-		end = start + size;
-		WARN_ON_ONCE(1);
-	}
+	if (WARN_ON_ONCE(!IS_ALIGNED(addr, PAGE_SIZE))) {
+		start &= PAGE_MASK;
+		end = PAGE_ALIGN(end);
+	}
 
 	if (start < MODULES_VADDR || start >= MODULES_END)
 		return -EINVAL;
 
-	if (end < MODULES_VADDR || start >= MODULES_END)
+	if (end < MODULES_VADDR || end >= MODULES_END)
 		return -EINVAL;
 
 	data.set_mask = set_mask;
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/