Re: [PATCH] arch:arm:mm:Correction in the boundary check for module end address.

From: Jungseung Lee
Date: Mon Nov 09 2015 - 10:36:05 EST


Hi,

2015-11-09 16:57 GMT+09:00 Shailendra Verma <shailendra.v@xxxxxxxxxx>:
> From: Shailendra Verma <Shailendra.v@xxxxxxxxxxx>
>
> The module end boundary check is not proper.The out of bound value
> of module end can produce undesired results.
>
> Signed-off-by: Shailendra Verma <Shailendra.v@xxxxxxxxxxx>
> Reviewed-by: Ravikant Bijendra Sharma <ravikant.s2@xxxxxxxxxxx>
> ---
> linux-4.3-rc6/arch/arm/mm/pageattr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/linux-4.3-rc6/arch/arm/mm/pageattr.c b/linux-4.3-rc6/arch/arm/mm/pageattr.c
> index cf30daf..be7fe4b 100644
> --- a/linux-4.3-rc6/arch/arm/mm/pageattr.c
> +++ b/linux-4.3-rc6/arch/arm/mm/pageattr.c
> @@ -52,7 +52,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
>
Same patch with proper format is already submitted by Hillf.
https://lkml.org/lkml/2015/5/3/202

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