[patch] ARM: fix module-bound check in setting page attributes

Hillf Danton hillf.zj at alibaba-inc.com
Sun May 3 19:52:31 PDT 2015


It was introduced in commit f2ca09f381a59
(ARM: 8311/1: Don't use is_module_addr in setting page attributes)

We have no need to check start twice, but see if end is also in range.

Signed-off-by: Hillf Danton <hillf.zj at alibaba-inc.com>
Acked-by: Laura Abbott <laura at labbott.name>
---

--- a/arch/arm/mm/pageattr.c	Mon May  4 10:33:49 2015
+++ b/arch/arm/mm/pageattr.c	Mon May  4 10:35:32 2015
@@ -52,7 +52,7 @@ static int change_memory_common(unsigned
 	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;
--





More information about the linux-arm-kernel mailing list