Re: PR/46973 CVS commit: src/sys/kern

[Wolfgang Stukenbrock <wolfgang.stukenbrock%nagler-company.com@localhost>]

Hi,

I'm not shure if this question is directed to Christos or to me.

The definition now returned is defined to 0, so no change in behaviour 
is done by this patch.

I think, that if it is still possible to pull it up without too much 
work, it should be done. But it is not critical if not.

From my side, the PR can be closed.
The correction will show up in future releases (-7 I think), and thats 
OK for me.

best regards

W. Stukenbrock

David Holland wrote:

> The following reply was made to PR kern/46973; it has been noted by GNATS.
>
> From: David Holland <dholland-bugs%netbsd.org@localhost>
> To: gnats-bugs%NetBSD.org@localhost
> Cc: 
> Subject: Re: PR/46973 CVS commit: src/sys/kern
> Date: Sun, 16 Sep 2012 18:04:43 +0000
>
>  On Sun, Sep 16, 2012 at 02:40:04PM +0000, Christos Zoulas wrote:
>   >  Modified Files:
>   >          src/sys/kern: kern_auth.c
>   >  
>   >  Log Message:
>   >  PR/46973: Dr. Wolfgang Stukenbrock: kauth_authorize_action_internal()
>   >  returns non-macro value as it should do
>  
>  Is there any reason to pull this up to -6, or can the PR be closed?
>  
>  -- 
>  David A. Holland
>  dholland%netbsd.org@localhost
>