On Thu, Jul 05, 2007 at 04:26:29PM +0000, [EMAIL PROTECTED] wrote:
> Author: vlendec
> Date: 2007-07-05 16:26:27 +0000 (Thu, 05 Jul 2007)
> New Revision: 23724
> 
> WebSVN: 
> http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=23724
> 
> Log:
> Reduce access to the global inbuf a tiny bit. Add a struct smb_request
> that contains some of the fields from the SMB header, removing the need
> to access inbuf directly. This right now is used only in the open file
> code & friends, and creating that header is only done when needed. This
> needs more work, but it is a start.
> 
> Jeremy, I'm only checking this into 3_0, please review before I merge it
> to _26.

+1 from me except the removal of this bit in smbd/nttrans.c. 
We need this as a placeholder to remember where to add the check
once we support SeSecurityPrivilege.

-#if 0
-       /* We need to support SeSecurityPrivilege for this. */
-       if ((access_mask & SEC_RIGHT_SYSTEM_SECURITY)) && 
-                       !user_has_privileges(current_user.nt_user_token,
-                               &se_security)) {
-               restore_case_semantics(conn, file_attributes);
-               return ERROR_NT(NT_STATUS_PRIVILEGE_NOT_HELD);
-       }
-#endif

Please merge to 3.0.26 !

Thanks,

        Jeremy.

Reply via email to