Bug 1387241
Summary: | Pass proper permission to acl_permit() in posix_acl_open() | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Jiffin <jthottan> |
Component: | access-control | Assignee: | Jiffin <jthottan> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | mainline | CC: | bugs |
Target Milestone: | --- | Keywords: | Triaged |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | glusterfs-3.10.0 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-03-06 17:30:52 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Jiffin
2016-10-20 12:29:09 UTC
REVIEW: http://review.gluster.org/15688 (access_control : Convert switch case into if .. else in posix_acl_open()) posted (#1) for review on master by jiffin tony Thottan (jthottan) REVIEW: http://review.gluster.org/15688 (access_control : Convert switch case into if .. else in posix_acl_open()) posted (#2) for review on master by jiffin tony Thottan (jthottan) REVIEW: http://review.gluster.org/15688 (access_control : address O_TRUNC and O_APPEND flag properly in posix_acl_open) posted (#3) for review on master by jiffin tony Thottan (jthottan) COMMIT: http://review.gluster.org/15688 committed in master by Kaleb KEITHLEY (kkeithle) ------ commit e81fd0b85c8dd3f521e54e32b7da2f99a513f2f2 Author: Jiffin Tony Thottan <jthottan> Date: Thu Nov 17 18:22:39 2016 +0530 access_control : address O_TRUNC and O_APPEND flag properly in posix_acl_open In posix_acl_open, in switch value passed is (flag & O_ACCMODE). The value for O_ACCMODE is 0003, so the result will always be less than or equal to 3. But value for O_TRUNC is 01000 and O_APPEND is 02000, so it is not right to check it in switch case Change-Id: Ia17db80a6a5f681c35e08e062d384f33ef7e0354 BUG: 1387241 Signed-off-by: Jiffin Tony Thottan <jthottan> Reviewed-on: http://review.gluster.org/15688 Smoke: Gluster Build System <jenkins.org> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Niels de Vos <ndevos> Reviewed-by: Kaleb KEITHLEY <kkeithle> REVIEW: http://review.gluster.org/16201 (access-control/tests : correct the test case bug-1387241.t) posted (#1) for review on master by jiffin tony Thottan (jthottan) This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.10.0, please open a new bug report. glusterfs-3.10.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://lists.gluster.org/pipermail/gluster-users/2017-February/030119.html [2] https://www.gluster.org/pipermail/gluster-users/ |