Re: CVS commit: [elad-kernelauth] src/sys

[YAMAMOTO Takashi <yamt%mwd.biglobe.ne.jp@localhost>]

> > its name doesn't imply anything about its usage.
> > generally, exporting more symbols is a bad idea, esp. when
> > its name is too generic like this.
> > 
> > i think it's better to make it static and
> > move process_authorize to kern_auth.c.
> 
> See attached diff, it:
> 
>   - changes builtin_{generic,process} to
>     kauth_builtin_scope_{generic,process}, and makes them static.
>   - remove extern for the above two from kauth.h.
>   - moves process_authorize() from kern_proc.c to kern_auth.c.
> 
> Is this okay?
> 
> -e.

looks fine to me.

YAMAMOTO Takashi