small se_access_check patch

Jeremy Allison jra at samba.org
Tue Sep 25 17:04:02 GMT 2001


On Mon, Sep 24, 2001 at 07:41:50PM +0200, Elrond wrote:
> 
> Hi,
> 
> These are some small changes to se_access_check.
> 
> Most notably, I changed the second parameter from
> struct current_user to NT_USER_TOKEN.
> 
> This requires any se_access_check to be changed like:
> 	se_access_check(..., user, ...)
> to:
> 	se_access_check(..., user->nt_user_token, ...)
> 
> In the event of user possibly being NULL (which would have
> coredumped se_access_check anyway), it should be changed
> to:
> 	se_access_check(..., (user ? user->nt_user_token : NULL), ...)
> 
> The const are needed by TNG specific things, and they
> should not hurt normal samba, because the only function
> outside util_seaccess.c, which is affected is sid_equal,
> which already has const in it.

Works for me - looks tidier too. I've committed this to 2.2.2 and HEAD..

Thanks,

	Jeremy.




More information about the samba-technical mailing list