Second part of bugfix for 6865 - acl_xattr module: Has dependency that inherit acls...

We also need dos filemode = true set as well.
Jeremy.

diff --git a/docs-xml/smbdotconf/misc/dosfilemode.xml b/docs-xml/smbdotconf/misc/dosfilemode.xml
index e67ccd935a58966940dcef9110beb3025d3282cc..cf531927337b3916d635f0b2f3ae14b43f1507ab 100644 (file)
--- a/docs-xml/smbdotconf/misc/dosfilemode.xml
+++ b/docs-xml/smbdotconf/misc/dosfilemode.xml
@@ -12,7 +12,13 @@
        (including ACL) on it. Note that a user belonging to the group
        owning the file will not be allowed to change permissions if
        the group is only granted read access. Ownership of the
-       file/directory may also be changed.</para>
+       file/directory may also be changed.
+
+       Note that using the VFS modules acl_xattr or acl_tdb which store native
+       Windows as meta-data will automatically turn this option on for any
+       share for which they are loaded, as they require this option to emulate
+       Windows ACLs correctly.
+       </para>
 </description>
 <value type="default">no</value>
 </samba:parameter>

diff --git a/source3/include/proto.h b/source3/include/proto.h
index 09933405eed6f3bf85d0f81c9024d7df61a67d6a..2dc0c83e022f8ce1edae7dd457605730b589993e 100644 (file)
--- a/source3/include/proto.h
+++ b/source3/include/proto.h
@@ -4374,7 +4374,6 @@ void lp_set_posix_default_cifsx_readwrite_locktype(enum brl_flavour val);
 int lp_min_receive_file_size(void);
 char* lp_perfcount_module(void);
 void lp_set_passdb_backend(const char *backend);
-bool set_inherit_acls(int i);
 
 /* The following definitions come from param/util.c  */
 

diff --git a/source3/modules/vfs_acl_tdb.c b/source3/modules/vfs_acl_tdb.c
index 3d06e520e87a3bd64a2b0e9c3d9f347337266502..db3881e1e738029982bc6730120e0f84da2afd6b 100644 (file)
--- a/source3/modules/vfs_acl_tdb.c
+++ b/source3/modules/vfs_acl_tdb.c
@@ -384,9 +384,11 @@ static int connect_acl_tdb(struct vfs_handle_struct *handle,
 
        /* Ensure we have "inherit acls = yes" if we're
         * using this module. */
-       DEBUG(2,("connect_acl_tdb: setting 'inherit acls = true' for service %s\n",
+       DEBUG(2,("connect_acl_tdb: setting 'inherit acls = true' "
+               "and 'dos filemode = true' for service %s\n",
                service ));
-       set_inherit_acls(SNUM(handle->conn));
+       lp_do_parameter(SNUM(handle->conn), "inherit acls", "true");
+       lp_do_parameter(SNUM(handle->conn), "dos filemode", "true");
 
        return 0;
 }

diff --git a/source3/modules/vfs_acl_xattr.c b/source3/modules/vfs_acl_xattr.c
index 11ca8902c09d1648637334e1e1bbab5704aa7e04..962d1b7b09dc954b10adaf1da3ce7dbbeca99596 100644 (file)
--- a/source3/modules/vfs_acl_xattr.c
+++ b/source3/modules/vfs_acl_xattr.c
@@ -214,9 +214,13 @@ static int connect_acl_xattr(struct vfs_handle_struct *handle,
 {
        /* Ensure we have "inherit acls = yes" if we're
         * using this module. */
-       DEBUG(2,("connect_acl_xattr: setting 'inherit acls = true' for service %s\n",
+       DEBUG(2,("connect_acl_xattr: setting 'inherit acls = true' "
+               "and 'dos filemode = true' for service %s\n",
                service ));
-       set_inherit_acls(SNUM(handle->conn));
+
+        lp_do_parameter(SNUM(handle->conn), "inherit acls", "true");
+        lp_do_parameter(SNUM(handle->conn), "dos filemode", "true");
+
        return 0;
 }
 

diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index 9a0f834f99a6dffd0b806ecd68619ea74c25596d..c62deb5eda5948e59ef214293423968114ea2310 100644 (file)
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -9843,13 +9843,3 @@ void lp_set_passdb_backend(const char *backend)
 {
        string_set(&Globals.szPassdbBackend, backend);
 }
-
-bool set_inherit_acls(int i)
-{
-       if (!LP_SNUM_OK(i)) {
-               return false;
-       }
-       ServicePtrs[(i)]->bInheritACLS = true;
-
-       return true;
-}