From owner-cvs-all Fri Aug 10 10:17:42 2001 Delivered-To: cvs-all@freebsd.org Received: from mail.gmx.net (pop.gmx.net [194.221.183.20]) by hub.freebsd.org (Postfix) with SMTP id C753937B403 for ; Fri, 10 Aug 2001 10:17:33 -0700 (PDT) (envelope-from tmoestl@gmx.net) Received: (qmail 32706 invoked by uid 0); 10 Aug 2001 17:17:31 -0000 Received: from p3e9e03b7.dip.t-dialin.net (HELO forge.local) (62.158.3.183) by mail.gmx.net (mp001-rz3) with SMTP; 10 Aug 2001 17:17:31 -0000 Received: from tmm by forge.local with local (Exim 3.30 #1) id 15VFua-0000dV-00; Fri, 10 Aug 2001 19:17:28 +0200 Date: Fri, 10 Aug 2001 19:17:28 +0200 From: Thomas Moestl To: =?iso-8859-1?Q?S=F8ren_Schmidt?= Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/ata atapi-cd.c Message-ID: <20010810191728.A2239@crow.dom2ip.de> Mail-Followup-To: =?iso-8859-1?Q?S=F8ren_Schmidt?= , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org References: <200107292101.f6TL1DM92117@freefall.freebsd.org> <200108101555.f7AFtVi22639@freebsd.dk> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit User-Agent: Mutt/1.2.5i In-Reply-To: <200108101555.f7AFtVi22639@freebsd.dk>; from sos@freebsd.dk on Fri, Aug 10, 2001 at 05:55:30PM +0200 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Fri, 2001/08/10 at 17:55:30 +0200, Søren Schmidt wrote: > It seems Thomas Moestl wrote: > > tmm 2001/07/29 14:01:13 PDT > > > > Modified files: > > sys/dev/ata atapi-cd.c > > Log: > > Disallow ATAPI CD transfers that are not a multiple of the device block > > size (previously, the transfer size would be rounded up to a multiple of > > the block size, which would overflow the buffer). > > This fixes panics when doing things like trying to mount audio CD's. > > > > PR: kern/21946 > > Review Timeout: sos > > Excuse me ???? > > I have not seen a single request for a review of this in my mailbox... > > Hmm.... Hmm. I have sent 3 mails to the address you sent this message from on this matter. Maybe they were lost somehow on the way? In any case, I apologize that I didn't wait for feedback, I just thought it might be good to have this patch in 4.4. - thomas To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message