From: Bram Moolenaar <Bram@vim.org>
Date: Thu, 14 Jul 2016 18:37:07 +0000 (+0200)
Subject: patch 7.4.2035
X-Git-Tag: v7.4.2035
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=da4127794aa333631bde879e73bcfce4aef42f85;p=vim

patch 7.4.2035
Problem:    On Solaris with ZFS the ACL may get removed.
Solution:   Always restore the ACL for Solaris ZFS. (Danek Duvall)
---

diff --git a/src/fileio.c b/src/fileio.c
index 14a6f31ba..235b412fa 100644
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -4709,9 +4709,17 @@ restore_backup:
     if (perm >= 0)		/* set perm. of new file same as old file */
 	(void)mch_setperm(wfname, perm);
 #ifdef HAVE_ACL
-    /* Probably need to set the ACL before changing the user (can't set the
-     * ACL on a file the user doesn't own). */
+    /*
+     * Probably need to set the ACL before changing the user (can't set the
+     * ACL on a file the user doesn't own).
+     * On Solaris, with ZFS and the aclmode property set to "discard" (the
+     * default), chmod() discards all part of a file's ACL that don't represent
+     * the mode of the file.  It's non-trivial for us to discover whether we're
+     * in that situation, so we simply always re-set the ACL.
+     */
+# ifndef HAVE_SOLARIS_ZFS_ACL
     if (!backup_copy)
+# endif
 	mch_set_acl(wfname, acl);
 #endif
 #ifdef FEAT_CRYPT
diff --git a/src/version.c b/src/version.c
index af29306e1..0dfb7593b 100644
--- a/src/version.c
+++ b/src/version.c
@@ -758,6 +758,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2035,
 /**/
     2034,
 /**/