Subject: Re: CVS commit: src/sys/dev/tc
To: Tom Spindler <dogcow@babymeat.com>
From: Elad Efrat <elad@NetBSD.org>
List: tech-kern
Date: 11/08/2006 12:16:33
Tom Spindler wrote:
>>> Log Message:
>>> remove extraneous kauth check; since open already does the auth - and
>>> mmap doesn't even get passed the lwp - the check here neither compiles
>>> nor does anything terribly useful.
>> the check replaced an existing check -- are you sure it's okay to leave
>> just the open one? (I remember looking into it in the past but I can't
>> remember what was the verdict)
>
> No, I'm not positive - but a quick skim of other devices that
> implemented mmap didn't have the check, either. (I also don't know
> how you'd grab the lwp, either - curproc?)
kauth_cred_get(), and myself not being familiar with the code I decided
to leave the check in unless someone who is can assure it's safe to
remove.
-e.
--
Elad Efrat