Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix NFS credential #4772

Closed
wants to merge 1 commit into from
Closed

Fix NFS credential #4772

wants to merge 1 commit into from

Commits on Jun 20, 2016

  1. Fix NFS credential

    The commit f74b821 caused a regression where creating file through NFS will
    always create a file owned by root. This is because the patch enables the KSID
    code in zfs_acl_ids_create, which it would use euid and egid of the current
    process. However, on Linux, we should use fsuid and fsgid for file operations,
    which is the original behaviour. So we revert this part of code.
    
    The patch also enables secpolicy_vnode_*, since they are also used in file
    operations, we change them to use fsuid and fsgid.
    
    Signed-off-by: Chunwei Chen <david.chen@osnexus.com>
    Chunwei Chen committed Jun 20, 2016
    Configuration menu
    Copy the full SHA
    2e9174f View commit details
    Browse the repository at this point in the history