On 02/13/2014 11:29 PM, Guangliang Zhao wrote: > Default ACL couldn't be inherited by the symlink in the > parent directory. This resolve it. This looks good to me, and it seems to be consistent with what other file systems do. Reviewed-by: Alex Elder <elder@xxxxxxxxxx> > Signed-off-by: Guangliang Zhao <lucienchao@xxxxxxxxx> > --- > fs/ceph/dir.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/fs/ceph/dir.c b/fs/ceph/dir.c > index 619616d..2650570 100644 > --- a/fs/ceph/dir.c > +++ b/fs/ceph/dir.c > @@ -735,6 +735,10 @@ static int ceph_symlink(struct inode *dir, struct dentry *dentry, > if (!err && !req->r_reply_info.head->is_dentry) > err = ceph_handle_notrace_create(dir, dentry); > ceph_mdsc_put_request(req); > + > + if (!err) > + err = ceph_init_acl(dentry, dentry->d_inode, dir); > + > if (err) > d_drop(dentry); > return err; > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
