[SCM] Samba Shared Repository - branch master updated - 6a4957d35d50e6508917aca62b282ae4904187c8

2008-11-25 Thread Michael Adam
The branch, master has been updated
   via  6a4957d35d50e6508917aca62b282ae4904187c8 (commit)
   via  afbfbd7f4c656fa4ed036314837024be8cd634c9 (commit)
   via  bf04324592695fd6e711ba25a89d47e1b61fa33e (commit)
  from  9d2c2a7a0e9e69c8fa2ce81af79007da0e32605b (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 6a4957d35d50e6508917aca62b282ae4904187c8
Author: Michael Adam [EMAIL PROTECTED]
Date:   Sun Nov 23 22:59:40 2008 +0100

UNFINISHED - s3:idmap_ad: multi-domain

Michael

commit afbfbd7f4c656fa4ed036314837024be8cd634c9
Author: Michael Adam [EMAIL PROTECTED]
Date:   Mon Nov 17 10:29:41 2008 +0100

[s3]zfsacl: return is not a function.

Michael

commit bf04324592695fd6e711ba25a89d47e1b61fa33e
Author: Nils Goroll [EMAIL PROTECTED]
Date:   Mon Nov 17 00:55:16 2008 +0100

[s3]zfsacl: Prevent calling POSIX ACL vfs methods on zfs share.

This is a proposed fix for Bugs #5135 and #5446.

Signed-off-by: Michael Adam [EMAIL PROTECTED]

---

Summary of changes:
 source3/modules/vfs_zfsacl.c |   83 +++
 source3/winbindd/idmap_ad.c  |  152 +++---
 2 files changed, 166 insertions(+), 69 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/modules/vfs_zfsacl.c b/source3/modules/vfs_zfsacl.c
index 3688b23..a5b0490 100644
--- a/source3/modules/vfs_zfsacl.c
+++ b/source3/modules/vfs_zfsacl.c
@@ -212,9 +212,92 @@ static NTSTATUS zfsacl_fset_nt_acl(vfs_handle_struct 
*handle,
return zfs_set_nt_acl(handle, fsp, security_info_sent, psd);
 }
 
+/* [EMAIL PROTECTED] 2008-06-16 :
+
+   See also
+   - https://bugzilla.samba.org/show_bug.cgi?id=5446
+   - http://bugs.opensolaris.org/view_bug.do?bug_id=6688240
+
+   Solaris supports NFSv4 and ZFS ACLs through a common system call, acl(2)
+   with ACE_SETACL / ACE_GETACL / ACE_GETACLCNT, which is being wrapped for
+   use by samba in this module.
+
+   As the acl(2) interface is identical for ZFS and for NFS, this module,
+   vfs_zfsacl, can not only be used for ZFS, but also for sharing NFSv4
+   mounts on Solaris.
+
+   But while traditional POSIX DRAFT ACLs (using acl(2) with SETACL
+   / GETACL / GETACLCNT) fail for ZFS, the Solaris NFS client
+   implemets a compatibility wrapper, which will make calls to
+   traditional ACL calls though vfs_solarisacl succeed. As the
+   compatibility wrapper's implementation is (by design) incomplete,
+   we want to make sure that it is never being called.
+
+   As long as Samba does not support an exiplicit method for a module
+   to define conflicting vfs methods, we should override all conflicting
+   methods here.
+
+   For this to work, we need to make sure that this module is initialised
+   *after* vfs_solarisacl
+
+   Function declarations taken from vfs_solarisacl
+*/
+
+SMB_ACL_T zfsacl_fail__sys_acl_get_file(vfs_handle_struct *handle,
+   const char *path_p,
+   SMB_ACL_TYPE_T type)
+{
+   return (SMB_ACL_T)NULL;
+}
+SMB_ACL_T zfsacl_fail__sys_acl_get_fd(vfs_handle_struct *handle,
+ files_struct *fsp,
+ int fd)
+{
+   return (SMB_ACL_T)NULL;
+}
+
+int zfsacl_fail__sys_acl_set_file(vfs_handle_struct *handle,
+ const char *name,
+ SMB_ACL_TYPE_T type,
+ SMB_ACL_T theacl)
+{
+   return -1;
+}
+
+int zfsacl_fail__sys_acl_set_fd(vfs_handle_struct *handle,
+   files_struct *fsp,
+   int fd, SMB_ACL_T theacl)
+{
+   return -1;
+}
+
+int zfsacl_fail__sys_acl_delete_def_file(vfs_handle_struct *handle,
+const char *path)
+{
+   return -1;
+}
+
 /* VFS operations structure */
 
 static vfs_op_tuple zfsacl_ops[] = {
+   /* invalidate conflicting VFS methods */
+   {SMB_VFS_OP(zfsacl_fail__sys_acl_get_file),
+SMB_VFS_OP_SYS_ACL_GET_FILE,
+SMB_VFS_LAYER_OPAQUE},
+   {SMB_VFS_OP(zfsacl_fail__sys_acl_get_fd),
+SMB_VFS_OP_SYS_ACL_GET_FD,
+SMB_VFS_LAYER_OPAQUE},
+   {SMB_VFS_OP(zfsacl_fail__sys_acl_set_file),
+SMB_VFS_OP_SYS_ACL_SET_FILE,
+SMB_VFS_LAYER_OPAQUE},
+   {SMB_VFS_OP(zfsacl_fail__sys_acl_set_fd),
+SMB_VFS_OP_SYS_ACL_SET_FD,
+SMB_VFS_LAYER_OPAQUE},
+   {SMB_VFS_OP(zfsacl_fail__sys_acl_delete_def_file),
+SMB_VFS_OP_SYS_ACL_DELETE_DEF_FILE,
+SMB_VFS_LAYER_OPAQUE},
+
+   /* actual methods */
{SMB_VFS_OP(zfsacl_fget_nt_acl), SMB_VFS_OP_FGET_NT_ACL,
 SMB_VFS_LAYER_OPAQUE},
{SMB_VFS_OP(zfsacl_get_nt_acl), SMB_VFS_OP_GET_NT_ACL,
diff --git 

Re: [SCM] Samba Shared Repository - branch master updated - 6a4957d35d50e6508917aca62b282ae4904187c8

2008-11-25 Thread Gerald (Jerry) Carter
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Michael Adam wrote:
 The branch, master has been updated
via  6a4957d35d50e6508917aca62b282ae4904187c8 (commit)
via  afbfbd7f4c656fa4ed036314837024be8cd634c9 (commit)
via  bf04324592695fd6e711ba25a89d47e1b61fa33e (commit)
   from  9d2c2a7a0e9e69c8fa2ce81af79007da0e32605b (commit)
 
 http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master
 
 
 - Log -
 commit 6a4957d35d50e6508917aca62b282ae4904187c8
 Author: Michael Adam [EMAIL PROTECTED]
 Date:   Sun Nov 23 22:59:40 2008 +0100
 
 UNFINISHED - s3:idmap_ad: multi-domain

Michael,

Why not just add the SFU schema support to the idmap_Adex plugin?
It already has global catalog and cross forest support?




cheers, jerry
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFJLAlpIR7qMdg1EfYRAk/FAJ9JO1yNGCKbdiSuz+bXcYOloxCj+ACgtzci
m+zd6UVg+P5d/b0RrvW9JNE=
=Vnu0
-END PGP SIGNATURE-