[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <00cd01d0767b$0cbe08a0$263a19e0$@alibaba-inc.com>
Date: Tue, 14 Apr 2015 14:20:05 +0800
From: "Hillf Danton" <hillf.zj@...baba-inc.com>
To: "Hillf Danton" <hillf.zj@...baba-inc.com>,
"'Laura Abbott'" <lauraa@...eaurora.org>
Cc: "'Russell King'" <rmk+kernel@....linux.org.uk>,
"'linux-kernel'" <linux-kernel@...r.kernel.org>,
<linux@....linux.org.uk>, "'Arnd Bergmann'" <arnd@...db.de>
Subject: Re: [patch] ARM: fix module-bound check in setting page attributes
Ping
> -----Original Message-----
> From: Hillf Danton [mailto:hillf.zj@...baba-inc.com]
> Sent: Tuesday, April 07, 2015 4:31 PM
> To: Hillf Danton; 'Laura Abbott'
> Cc: 'Russell King'; 'linux-kernel'; linux@....linux.org.uk
> Subject: Re: [patch] ARM: fix module-bound check in setting page attributes
>
> Ping
> >
> > 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@...baba-inc.com>
> > ---
> >
> > --- a/arch/arm/mm/pageattr.c Wed Mar 25 11:55:13 2015
> > +++ b/arch/arm/mm/pageattr.c Wed Mar 25 11:57:31 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;
> > --
--
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/
Powered by blists - more mailing lists