Re: svn commit: samba r23724 - in branches/SAMBA_3_0/source: include printing rpc_server smbd
On Thu, Jul 05, 2007 at 04:26:29PM +, [EMAIL PROTECTED] wrote: Author: vlendec Date: 2007-07-05 16:26:27 + (Thu, 05 Jul 2007) New Revision: 23724 WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=revroot=sambarev=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.
svn commit: samba r23724 - in branches/SAMBA_3_0/source: include printing rpc_server smbd
Author: vlendec Date: 2007-07-05 16:26:27 + (Thu, 05 Jul 2007) New Revision: 23724 WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=revroot=sambarev=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. Volker Modified: branches/SAMBA_3_0/source/include/smb.h branches/SAMBA_3_0/source/printing/nt_printing.c branches/SAMBA_3_0/source/rpc_server/srv_srvsvc_nt.c branches/SAMBA_3_0/source/smbd/dir.c branches/SAMBA_3_0/source/smbd/nttrans.c branches/SAMBA_3_0/source/smbd/open.c branches/SAMBA_3_0/source/smbd/posix_acls.c branches/SAMBA_3_0/source/smbd/process.c branches/SAMBA_3_0/source/smbd/reply.c branches/SAMBA_3_0/source/smbd/trans2.c Changeset: Sorry, the patch is too large (1197 lines) to include; please use WebSVN to see it! WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=revroot=sambarev=23724
Re: svn commit: samba r23724 - in branches/SAMBA_3_0/source: include printing rpc_server smbd
On Thu, Jul 05, 2007 at 04:26:29PM +, [EMAIL PROTECTED] wrote: Jeremy, I'm only checking this into 3_0, please review before I merge it to _26. Not a good day for reviews today I'm afraid. This might have to wait until the weekend, sorry. Jeremy.