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