From: Mark Harmstone <[email protected]> On Behalf Of Mark Harmstone
Sent: Friday, September 18, 2020 9:28 PM
> 
> Could I suggest that system.ntfs_security be renamed to security.NTACL? That's
> what WinBtrfs uses, and it means you'd be able to create a working Btrfs copy
> of a Windows installation just by using rsync. IIRC Samba also understands
> xattrs called security.NTACL, when you've put it in the right mode.
> 
> Quite apart from anything else, it's an NT security descriptor, not 
> specifically
> NTFS - I'm fairly sure ReFS uses the same format (for what it's worth).
> 

Hi Mark! Seems reasonable. Will be changed in V8.

> On 18/9/20 5:21 pm, Konstantin Komarov wrote:
> > This adds attrib operations
> >
> > Signed-off-by: Konstantin Komarov <[email protected]>
> > ---
> >  fs/ntfs3/attrib.c   | 1312 +++++++++++++++++++++++++++++++++++++++++++
> >  fs/ntfs3/attrlist.c |  462 +++++++++++++++
> >  fs/ntfs3/xattr.c    | 1041 ++++++++++++++++++++++++++++++++++
> >  3 files changed, 2815 insertions(+)
> >  create mode 100644 fs/ntfs3/attrib.c
> >  create mode 100644 fs/ntfs3/attrlist.c
> >  create mode 100644 fs/ntfs3/xattr.c
[]
> > +   if (err < 0)
> > +           goto out1;
> > +   if (!err) {
> > +           posix_acl_release(acl);
> > +           acl = NULL;
> > +   }
> > +
> > +   if (!S_ISDIR(inode->i_mode)) {
> > +           posix_acl_release(default_acl);
> > +           default_acl = NULL;
> > +   }
> > +
> > +   if (default_acl)
> > +           err = ntfs_set_acl_ex(inode, default_acl, ACL_TYPE_DEFAULT, 1);
> > +
> > +   if (!acl)
> > +           inode->i_acl = NULL;
> > +   else if (!err)
> > +           err = ntfs_set_acl_ex(inode, acl, ACL_TYPE_ACCESS, 1);
> > +
> > +   posix_acl_release(acl);
> > +out1:
> > +   posix_acl_release(default_acl);
> > +
> > +out:
> > +   return err;
> > +}
> > +
> > +static bool ntfs_xattr_user_list(struct dentry *dentry)
> > +{
> > +   return 1;
> > +}
> > +
> > +static const struct xattr_handler ntfs_xattr_handler = {
> > +   .prefix = "",
> > +   .get = ntfs_getxattr,
> > +   .set = ntfs_setxattr,
> > +   .list = ntfs_xattr_user_list,
> > +};
> > +
> > +const struct xattr_handler *ntfs_xattr_handlers[] = { &ntfs_xattr_handler,
> > +                                                 NULL };
> 

Thanks.

Reply via email to