Bug 1238149

Summary:	FSAL_GLUSTER : avoid possible memory corruption for inherit acl
Product:	[Red Hat Storage] Red Hat Gluster Storage	Reporter:	Jiffin <jthottan>
Component:	nfs-ganesha	Assignee:	Jiffin <jthottan>
Status:	CLOSED ERRATA	QA Contact:	Saurabh <saujain>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rhgs-3.1	CC:	akhakhar, annair, ansubram, asrivast, kkeithle, mmadhusu, mzywusko, ndevos, nlevinki, skoduri, vagarwal
Target Milestone:	---
Target Release:	RHGS 3.1.0
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	nfs-ganesha-2.2.0-4	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1238145	Environment:
Last Closed:	2015-07-29 05:08:58 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:
Bug Depends On:	1238145
Bug Blocks:	1202842

Description Jiffin 2015-07-01 09:47:20 UTC

+++ This bug was initially created as a clone of Bug #1238145 +++

Description of problem:

fsal_acl_t entries which represents inherit acl may not contains all the required entries, but fsal_acl_2_posix_acl() api include accordingly.For that we may need to allocate acl_t structure with more than the required entries.

Comment 4 Jiffin 2015-07-02 08:31:09 UTC

This is fixed in https://review.gerrithub.io/#/c/237728/

Comment 6 Saurabh 2015-07-14 10:56:21 UTC

As per discussion with developer, the memory corruption could have happened in case of inherit acl setfacl and getfacl, but post fix I can see that setting and getting acls with inherit flags works fine.

Comment 7 errata-xmlrpc 2015-07-29 05:08:58 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHSA-2015-1495.html