From: Shailendra Verma <shailendra.v@samung.com>
To: Russell King <linux@arm.linux.org.uk>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        =?UTF-8?q?Arve=20Hj=C3=B8nnev=C3=A5g?= <arve@android.com>,
        Riley Andrews <riandrews@android.com>,
        Laura Abbott <lauraa@codeaurora.org>,
        Jungseung Lee <js07.lee@gmail.com>,
        Shailendra Verma <shailendra.capricorn@gmail.com>,
        Dan Carpenter <dan.carpenter@oracle.com>,
        Mitchel Humpherys <mitchelh@codeaurora.org>,
        Sumit Semwal <sumit.semwal@linaro.org>,
        "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>,
        Gioh Kim <gioh.kim@lge.com>,
        Rasmus Villemoes <linux@rasmusvillemoes.dk>,
        Shawn Lin <shawn.lin@rock-chips.com>,
        Markus Elfring <elfring@users.sourceforge.net>,
        linux-arm-kernel@lists.infradead.org, devel@driverdev.osuosl.org
Cc: Shailendra Verma <shailendra.v@samsung.com>,
        linux-kernel@vger.kernel.org, vidushi.koul@samsung.com
Subject: [PATCH] Arch: arm: mm: Aligning the module end and Correction in
Date: Thu, 10 Nov 2016 15:25:34 +0530
Message-id: <1478771734-1464-1-git-send-email-shailendra.v@samung.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1229
Lines: 40

From: "Shailendra Verma" <shailendra.v@samsung.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@alibaba-inc.com>
Signed-off-by: Shailendra Verma <Shailendra.v@samsung.com>
---
 arch/arm/mm/pageattr.c |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/arch/arm/mm/pageattr.c b/arch/arm/mm/pageattr.c
index d19b1ad..dd0b8a2 100644
--- a/arch/arm/mm/pageattr.c
+++ b/arch/arm/mm/pageattr.c
@@ -43,10 +43,9 @@ static int change_memory_common(unsigned long addr, int numpages,
 	int ret;
 	struct page_change_data data;
 
-	if (!IS_ALIGNED(addr, PAGE_SIZE)) {
+	if (WARN_ON_ONCE(!IS_ALIGNED(addr, PAGE_SIZE))) {
 		start &= PAGE_MASK;
-		end = start + size;
-		WARN_ON_ONCE(1);
+		end = PAGE_ALIGN(end);
 	}
 
 	if (!numpages)
@@ -55,7 +54,7 @@ static int change_memory_common(unsigned long addr, int numpages,
 	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