xfs
[Top] [All Lists]

Re: [PATCH v27 02/21] vfs: Add MAY_CREATE_FILE and MAY_CREATE_DIR permis

To: Andreas Gruenbacher <agruenba@xxxxxxxxxx>
Subject: Re: [PATCH v27 02/21] vfs: Add MAY_CREATE_FILE and MAY_CREATE_DIR permission flags
From: Miklos Szeredi <miklos@xxxxxxxxxx>
Date: Fri, 2 Dec 2016 10:22:58 +0100
Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, "Theodore Ts'o" <tytso@xxxxxxx>, Andreas Dilger <adilger.kernel@xxxxxxxxx>, "J. Bruce Fields" <bfields@xxxxxxxxxxxx>, Jeff Layton <jlayton@xxxxxxxxxxxxxxx>, Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx>, Anna Schumaker <anna.schumaker@xxxxxxxxxx>, Dave Chinner <david@xxxxxxxxxxxxx>, linux-ext4@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, LKML <linux-kernel@xxxxxxxxxxxxxxx>, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, linux-nfs@xxxxxxxxxxxxxxx, linux-cifs@xxxxxxxxxxxxxxx, linux-api@xxxxxxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=szeredi.hu; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=8RMSjYbZupBP1BlGGHA+aaEIFI67ApWU+R/XovtEhZs=; b=fFVS7R2YUBkn8ljOnr5aItQtlf8/uBpNV72rFUACp5bFut9vPhKkRezD3OQGEuxvul pinxeNCbAsxX22NB+wZl4P5Z1hPOjF5HxR2F0IifEBvdrQ+s2DwckjKiL/r5PeTwGqs7 9t43x7IpdzUqhzunoxi1yr3R6iCc4xRYTWFDA=
In-reply-to: <1476190256-1677-3-git-send-email-agruenba@xxxxxxxxxx>
References: <1476190256-1677-1-git-send-email-agruenba@xxxxxxxxxx> <1476190256-1677-3-git-send-email-agruenba@xxxxxxxxxx>
On Tue, Oct 11, 2016 at 2:50 PM, Andreas Gruenbacher
<agruenba@xxxxxxxxxx> wrote:
> Richacls distinguish between creating non-directories and directories. To
> support that, add an isdir parameter to may_create(). When checking
> inode_permission() for create permission, pass in an additional
> MAY_CREATE_FILE or MAY_CREATE_DIR mask flag.
>
> Add may_replace() to allow checking for delete and create access when
> replacing an existing file in vfs_rename().

That works for plain rename, but not for RENAME_EXCHANGE: is_dir and
new_is_dir may be different, which means we can be deleting a
non-directory *and* creating a directory in one dir and vice versa in
the other.

Thanks,
Miklos

<Prev in Thread] Current Thread [Next in Thread>
  • Re: [PATCH v27 02/21] vfs: Add MAY_CREATE_FILE and MAY_CREATE_DIR permission flags, Miklos Szeredi <=