[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <alpine.LFD.2.00.1210031544080.20211@dhcp-1-104.brq.redhat.com>
Date: Wed, 3 Oct 2012 15:46:54 +0200 (CEST)
From: Lukáš Czerner <lczerner@...hat.com>
To: Douglas Gilbert <dgilbert@...erlog.com>
cc: "Martin K. Petersen" <martin.petersen@...cle.com>,
Lukas Czerner <lczerner@...hat.com>,
linux-scsi@...r.kernel.org, JBottomley@...allels.com,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] scsi_debug: Fix off-by-one bug when unmapping region
On Wed, 5 Sep 2012, Douglas Gilbert wrote:
> Date: Wed, 05 Sep 2012 15:41:13 -0400
> From: Douglas Gilbert <dgilbert@...erlog.com>
> To: Martin K. Petersen <martin.petersen@...cle.com>
> Cc: Lukas Czerner <lczerner@...hat.com>, linux-scsi@...r.kernel.org,
> JBottomley@...allels.com, tytso@....edu, pbonzini@...hat.com,
> linux-kernel@...r.kernel.org
> Subject: Re: [PATCH] scsi_debug: Fix off-by-one bug when unmapping region
>
> On 12-08-17 12:11 PM, Martin K. Petersen wrote:
> > > > > > > "Lukas" == Lukas Czerner <lczerner@...hat.com> writes:
> >
> > Lukas> Currently it is possible to unmap one more block than user
> > Lukas> requested to due to the off-by-one error in unmap_region(). This
> > Lukas> is probably due to the fact that the end variable despite its
> > Lukas> name actually points to the last block to unmap + 1. However in
> > Lukas> the condition it is handled as the last block of the region to
> > Lukas> unmap.
> >
> > Acked-by: Martin K. Petersen <martin.petersen@...cle.com>
>
> Acked-by: Douglas Gilbert <dgilbert@...erlog.com>
James, could you please pick this one ?
Thanks!
-Lukas
--
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