>From Frank Filz <ffilz...@mindspring.com>: Frank Filz has uploaded a new change for review.
https://review.gerrithub.io/297897 Change subject: FSAL_VFS: Assure correct permission checking in open2 ...................................................................... FSAL_VFS: 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 openat system 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. Note that if FSAL_VFS is supporting ACLs, especially via the in memory only version, the permission check done by openat may not be correct. Change-Id: Ib20bfdc30a212f2755ac9efcb4bb4256dc8bc497 Signed-off-by: Frank S. Filz <ffilz...@mindspring.com> --- M src/FSAL/FSAL_VFS/file.c 1 file changed, 43 insertions(+), 10 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/97/297897/1 -- To view, visit https://review.gerrithub.io/297897 To unsubscribe, visit https://review.gerrithub.io/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib20bfdc30a212f2755ac9efcb4bb4256dc8bc497 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
