From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964912AbcAYShp (ORCPT ); Mon, 25 Jan 2016 13:37:45 -0500 Received: from mx1.redhat.com ([209.132.183.28]:37981 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933660AbcAYShm (ORCPT ); Mon, 25 Jan 2016 13:37:42 -0500 Subject: Re: [PATCH 1/4] arm: Fix wrong bounds check. To: mika.penttila@nextfour.com, linux-kernel@vger.kernel.org References: <1453561543-14756-1-git-send-email-mika.penttila@nextfour.com> <1453561543-14756-2-git-send-email-mika.penttila@nextfour.com> Cc: linux-mm@kvack.org, rientjes@google.com, linux@arm.linux.org.uk From: Laura Abbott Message-ID: <56A66B74.1060103@redhat.com> Date: Mon, 25 Jan 2016 10:37:40 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <1453561543-14756-2-git-send-email-mika.penttila@nextfour.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/23/2016 07:05 AM, mika.penttila@nextfour.com wrote: > From: Mika Penttilä > > Not related to this oops, but while at it, fix incorrect bounds check. > > Signed-off-by: Mika Penttilä mika.penttila@nextfour.com > > --- > arch/arm/mm/pageattr.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/mm/pageattr.c b/arch/arm/mm/pageattr.c > index cf30daf..be7fe4b 100644 > --- a/arch/arm/mm/pageattr.c > +++ b/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; > This has been submitted a few times before, not sure if it is pending in Russell's patch tracker or nobody has actually submitted it to the patch tracker. Russell, is this pending somewhere already? Thanks, Laura