summaryrefslogtreecommitdiffstats
path: root/xlators/system/posix-acl
diff options
context:
space:
mode:
authorAmar Tumballi <amarts@redhat.com>2012-07-26 17:16:00 +0530
committerAnand Avati <avati@redhat.com>2012-07-27 21:39:08 -0700
commitb4dbbe007da003962361c9a2579d1114f71619cc (patch)
tree99a31b94ee2289860dae1d06f0048bf94fb4d362 /xlators/system/posix-acl
parent07c90f84d3cb11fd9539008f3a95c49d088e62bc (diff)
acl: enable handling of FMODE_EXEC flag
on linux systems, with open(), we can get below flag as per 'linux/fs.h'. /* File is opened for execution with sys_execve / sys_uselib */ Instead of adding '#include <linux/fs.h>, its better to copy this absolute number into other variable because then we have to deal with declaring fmode_t etc etc.. With the fix, we can handle the file with '0711' permissions in the same way as backend linux filesystems. Change-Id: Ib1097fc0d2502af89c92d561eb4123cba15713f5 Signed-off-by: Amar Tumballi <amarts@redhat.com> Reviewed-on: http://review.gluster.com/3739 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Niels de Vos <ndevos@redhat.com> Tested-by: Niels de Vos <ndevos@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/system/posix-acl')
-rw-r--r--xlators/system/posix-acl/src/posix-acl.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/xlators/system/posix-acl/src/posix-acl.c b/xlators/system/posix-acl/src/posix-acl.c
index 0962f64044d..f54e20a6970 100644
--- a/xlators/system/posix-acl/src/posix-acl.c
+++ b/xlators/system/posix-acl/src/posix-acl.c
@@ -919,6 +919,13 @@ posix_acl_open (call_frame_t *frame, xlator_t *this, loc_t *loc, int flags,
switch (flags & O_ACCMODE) {
case O_RDONLY:
perm = POSIX_ACL_READ;
+
+ /* If O_FMODE_EXEC is present, its good enough
+ to have '--x' perm, and its not covered in
+ O_ACCMODE bits */
+ if (flags & O_FMODE_EXEC)
+ perm = POSIX_ACL_EXECUTE;
+
break;
case O_WRONLY:
case O_APPEND: