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 <el...@linaro.org>

> Signed-off-by: Guangliang Zhao <lucienc...@gmail.com>
> ---
>  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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to