• Chuck Lever's avatar
    NFSD: Support write delegations in LAYOUTGET · abc02e56
    Chuck Lever authored
    I noticed LAYOUTGET(LAYOUTIOMODE4_RW) returning NFS4ERR_ACCESS
    unexpectedly. The NFS client had created a file with mode 0444, and
    the server had returned a write delegation on the OPEN(CREATE). The
    client was requesting a RW layout using the write delegation stateid
    so that it could flush file modifications.
    
    Creating a read-only file does not seem to be problematic for
    NFSv4.1 without pNFS, so I began looking at NFSD's implementation of
    LAYOUTGET.
    
    The failure was because fh_verify() was doing a permission check as
    part of verifying the FH presented during the LAYOUTGET. It uses the
    loga_iomode value to specify the @accmode argument to fh_verify().
    fh_verify(MAY_WRITE) on a file whose mode is 0444 fails with -EACCES.
    
    To permit LAYOUT* operations in this case, add OWNER_OVERRIDE when
    checking the access permission of the incoming file handle for
    LAYOUTGET and LAYOUTCOMMIT.
    
    Cc: Christoph Hellwig <hch@lst.de>
    Cc: stable@vger.kernel.org # v6.6+
    Message-Id: 4E9C0D74-A06D-4DC3-A48A-73034DC40395@oracle.com
    Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
    Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
    Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
    abc02e56
nfs4proc.c 101 KB