>From Frank Filz <ffilz...@mindspring.com>:

Frank Filz has uploaded a new change for review.

  https://review.gerrithub.io/297900

Change subject: FSAL_GPFS: Assure correct permission checking in open2
......................................................................

FSAL_GPFS: Assure correct permission checking in open2

If open2 is by object, not by name, open2 does no permission checking.

If open2 is by name and was a create, the GPFSFSAL_create2 call was made
with caller credentials and thus was permission checked. A NO_CREATE
call to just open the file was not permission checked, so assure that
the caller (open2_by_name) does a permission check.

The way open2 is called, there is actually no functional difference
with this patch, however it should clarify things.

Change-Id: If90518a94397aa17ada592cb43e9b0c658524efb
Signed-off-by: Frank S. Filz <ffilz...@mindspring.com>
---
M src/FSAL/FSAL_GPFS/file.c
1 file changed, 11 insertions(+), 5 deletions(-)


  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/00/297900/1
-- 
To view, visit https://review.gerrithub.io/297900
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If90518a94397aa17ada592cb43e9b0c658524efb
Gerrit-PatchSet: 1
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Owner: Frank Filz <ffilz...@mindspring.com>

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to