[SCM] Samba Shared Repository - branch master updated

Volker Lendecke vlendec at samba.org
Tue May 15 10:41:02 UTC 2018


The branch, master has been updated
       via  506c520 smbd: fileserver: Change defaults to work with EA support out of the box.
      from  9fbd467 lib: Hold at most 10 outstanding paged result cookies

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


- Log -----------------------------------------------------------------
commit 506c520503eacff33064c1c23a068399f7296d86
Author: Jeremy Allison <jra at samba.org>
Date:   Mon May 14 11:09:53 2018 -0700

    smbd: fileserver: Change defaults to work with EA support out of the box.
    
    Signed-off-by: Jeremy Allison <jra at samba.org>
    Reviewed-by: Volker Lendecke <vl at samba.org>
    
    Autobuild-User(master): Volker Lendecke <vl at samba.org>
    Autobuild-Date(master): Tue May 15 12:40:48 CEST 2018 on sn-devel-144

-----------------------------------------------------------------------

Summary of changes:
 WHATSNEW.txt                                        | 9 ++++++++-
 docs-xml/smbdotconf/filename/mapreadonly.xml        | 9 ++++++++-
 docs-xml/smbdotconf/filename/storedosattributes.xml | 5 ++++-
 docs-xml/smbdotconf/protocol/easupport.xml          | 5 ++++-
 lib/param/loadparm.c                                | 6 +++++-
 source3/param/loadparm.c                            | 6 +++---
 6 files changed, 32 insertions(+), 8 deletions(-)


Changeset truncated at 500 lines:

diff --git a/WHATSNEW.txt b/WHATSNEW.txt
index 7bd3792..574e9b2 100644
--- a/WHATSNEW.txt
+++ b/WHATSNEW.txt
@@ -61,9 +61,16 @@ REMOVED FEATURES
 smb.conf changes
 ================
 
+As the most popular Samba install platforms (Linux and FreeBSD) both
+support extended attributes by default, the parameters "map readonly",
+"store dos attributes" and "ea support" have had their defaults changed
+to allow better Windows fileserver compatibility in a default install.
+
   Parameter Name                     Description             Default
   --------------                     -----------             -------
-
+  map readonly                       Default changed              no
+  store dos attributes               Default changed             yes
+  ea support                         Default changed             yes
 
 KNOWN ISSUES
 ============
diff --git a/docs-xml/smbdotconf/filename/mapreadonly.xml b/docs-xml/smbdotconf/filename/mapreadonly.xml
index 54a5e01..dae17c2 100644
--- a/docs-xml/smbdotconf/filename/mapreadonly.xml
+++ b/docs-xml/smbdotconf/filename/mapreadonly.xml
@@ -49,6 +49,13 @@
         attribute.
         </para>
 
+        <para>
+        The default has changed to no in Samba release 4.9.0 and above to allow better Windows
+        fileserver compatibility in a default install. In addition the default setting of
+        <smbconfoption name="store dos attributes"/> has been changed to <constant>Yes</constant>
+        in Samba release 4.9.0 and above.
+        </para>
+
 </description>
-<value type="default">yes</value>
+<value type="default">no</value>
 </samba:parameter>
diff --git a/docs-xml/smbdotconf/filename/storedosattributes.xml b/docs-xml/smbdotconf/filename/storedosattributes.xml
index 30665eb..2cdaeef 100644
--- a/docs-xml/smbdotconf/filename/storedosattributes.xml
+++ b/docs-xml/smbdotconf/filename/storedosattributes.xml
@@ -21,7 +21,10 @@
 	extended attribute by earlier versions of Samba, but they will not be able to read the create
 	time stored there. Storing the create time separately from the normal filesystem meta-data
 	allows Samba to faithfully reproduce NTFS semantics on top of a POSIX filesystem.
+
+	The default has changed to yes in Samba release 4.9.0 and above to allow better Windows
+	fileserver compatibility in a default install.
 	</para>
 </description>
-<value type="default">no</value>
+<value type="default">yes</value>
 </samba:parameter>
diff --git a/docs-xml/smbdotconf/protocol/easupport.xml b/docs-xml/smbdotconf/protocol/easupport.xml
index b453b86..403e48f 100644
--- a/docs-xml/smbdotconf/protocol/easupport.xml
+++ b/docs-xml/smbdotconf/protocol/easupport.xml
@@ -24,8 +24,11 @@
     access to this tight space via extended attribute support could consume all
     of it by unsuspecting client applications, which would prevent changing
     system metadata due to lack of space.
+
+    The default has changed to yes in Samba release 4.9.0 and above to allow better Windows
+    fileserver compatibility in a default install.
     </para>
 </description>
 
-<value type="default">no</value>
+<value type="default">yes</value>
 </samba:parameter>
diff --git a/lib/param/loadparm.c b/lib/param/loadparm.c
index bcb4141..3b7f805 100644
--- a/lib/param/loadparm.c
+++ b/lib/param/loadparm.c
@@ -2880,7 +2880,7 @@ struct loadparm_context *loadparm_init(TALLOC_CTX *mem_ctx)
 
 	lpcfg_do_global_parameter(lp_ctx, "strict sync", "yes");
 
-	lpcfg_do_global_parameter(lp_ctx, "map readonly", "yes");
+	lpcfg_do_global_parameter(lp_ctx, "map readonly", "no");
 
 	lpcfg_do_global_parameter(lp_ctx, "allow trusted domains", "yes");
 
@@ -3000,6 +3000,10 @@ struct loadparm_context *loadparm_init(TALLOC_CTX *mem_ctx)
 
 	lpcfg_do_global_parameter(lp_ctx, "check parent directory delete on close", "no");
 
+	lpcfg_do_global_parameter(lp_ctx, "ea support", "yes");
+
+	lpcfg_do_global_parameter(lp_ctx, "store dos attributes", "yes");
+
 	for (i = 0; parm_table[i].label; i++) {
 		if (!(lp_ctx->flags[i] & FLAG_CMDLINE)) {
 			lp_ctx->flags[i] |= FLAG_DEFAULT;
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index 520d066..ecff22e 100644
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -193,7 +193,7 @@ static const struct loadparm_service _sDefault =
 	.map_system = false,
 	.map_hidden = false,
 	.map_archive = true,
-	.store_dos_attributes = false,
+	.store_dos_attributes = true,
 	.dmapi_support = false,
 	.locking = true,
 	.strict_locking = Auto,
@@ -231,7 +231,7 @@ static const struct loadparm_service _sDefault =
 	._use_sendfile = false,
 	.map_acl_inherit = false,
 	.afs_share = false,
-	.ea_support = false,
+	.ea_support = true,
 	.acl_check_permissions = true,
 	.acl_map_full_control = true,
 	.acl_group_control = false,
@@ -239,7 +239,7 @@ static const struct loadparm_service _sDefault =
 	.allocation_roundup_size = SMB_ROUNDUP_ALLOCATION_SIZE,
 	.aio_read_size = 1,
 	.aio_write_size = 1,
-	.map_readonly = MAP_READONLY_YES,
+	.map_readonly = MAP_READONLY_NO,
 	.directory_name_cache_size = 100,
 	.smb_encrypt = SMB_SIGNING_DEFAULT,
 	.kernel_share_modes = true,


-- 
Samba Shared Repository



More information about the samba-cvs mailing list