summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristof Schmitt <cs@samba.org>2019-07-09 12:04:35 -0700
committerKarolin Seeger <kseeger@samba.org>2019-08-26 10:23:29 +0000
commitb3560baaf99886fcd55c5a03489ca03d82573099 (patch)
tree2b97fd84fdb09ba6f98e7f1ccf7dab8bb88afa59
parent548cc5183e471cb648ba7975e16f2a15ea78919e (diff)
downloadsamba-b3560baaf99886fcd55c5a03489ca03d82573099.tar.gz
vfs_gpfs: Remove merge_writeappend parameter
All supported GPFS versions now support setting WRITE and APPEND in the ACLs independently. Remove this now unused parameter to simplify the code. BUG: https://bugzilla.samba.org/show_bug.cgi?id=14032 Signed-off-by: Christof Schmitt <cs@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org> (cherry picked from commit 0aca678fcf1788a76cf0ff11399211c795aa7d2f)
-rw-r--r--source3/modules/vfs_gpfs.c23
1 files changed, 0 insertions, 23 deletions
diff --git a/source3/modules/vfs_gpfs.c b/source3/modules/vfs_gpfs.c
index d733df0aa5c..63aeeddf797 100644
--- a/source3/modules/vfs_gpfs.c
+++ b/source3/modules/vfs_gpfs.c
@@ -709,29 +709,6 @@ static struct gpfs_acl *vfs_gpfs_smbacl2gpfsacl(TALLOC_CTX *mem_ctx,
gace->aceType = aceprop->aceType;
gace->aceFlags = aceprop->aceFlags;
gace->aceMask = aceprop->aceMask;
-
- /*
- * GPFS can't distinguish between WRITE and APPEND on
- * files, so one being set without the other is an
- * error. Sorry for the many ()'s :-)
- */
-
- if (!fsp->is_directory
- &&
- ((((gace->aceMask & ACE4_MASK_WRITE) == 0)
- && ((gace->aceMask & ACE4_MASK_APPEND) != 0))
- ||
- (((gace->aceMask & ACE4_MASK_WRITE) != 0)
- && ((gace->aceMask & ACE4_MASK_APPEND) == 0)))
- &&
- lp_parm_bool(fsp->conn->params->service, "gpfs",
- "merge_writeappend", True)) {
- DEBUG(2, ("vfs_gpfs.c: file [%s]: ACE contains "
- "WRITE^APPEND, setting WRITE|APPEND\n",
- fsp_str_dbg(fsp)));
- gace->aceMask |= ACE4_MASK_WRITE|ACE4_MASK_APPEND;
- }
-
gace->aceIFlags = (aceprop->flags&SMB_ACE4_ID_SPECIAL) ? ACE4_IFLAG_SPECIAL_ID : 0;
if (aceprop->flags&SMB_ACE4_ID_SPECIAL)