Subject: Re: bounds checking RAW_PART [was: Re: CVS commit: src/sys/dev]
To: Jason R Thorpe <thorpej@wasabisystems.com>
From: Manuel Bouyer <bouyer@antioche.eu.org>
List: tech-kern
Date: 01/27/2003 22:22:52
On Sun, Jan 26, 2003 at 04:21:07PM -0800, Jason R Thorpe wrote:
> On Mon, Jan 27, 2003 at 01:15:39AM +0100, Christoph Badura wrote:
>
> > No Unix that I've used in the last 20 years has done this and I've never
> > had a need to read beyond the end of the raw partition. And if I suddenly
> > felt that urge, I'd simply change the disklabel to include more sectors.
> > There's nothing that prevents you from doing that.
> >
> > Note also that a number of drivers has been bounds checking RAW_PART forever
> > and nobody seems to complain.
>
> I have no problem with bounds-checking RAW_PART. But it should not be
> done with bound_check_with_label(). That function is desinged to bounds-
> check against the disklabel. RAW_PART is not to be checked against the
> disklabel. That is my only objection.
I don't think it should be done for SCSI disks and CD, which reports EOM fine.
Something needs to be done for IDE, and I think I've found how.
--
Manuel Bouyer <bouyer@antioche.eu.org>
NetBSD: 24 ans d'experience feront toujours la difference
--