linux-ext4 - Re: [PATCH v2] create_inode: set xattrs to the root directory as well

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 4 Sep 2020 13:43:18 -0600
From:   Andreas Dilger <adilger@...ger.ca>
To:     Antoine Tenart <antoine.tenart@...tlin.com>
Cc:     Theodore Ts'o <tytso@....edu>,
        Ext4 Developers List <linux-ext4@...r.kernel.org>,
        matthew.weber@...kwellcollins.com, thomas.petazzoni@...tlin.com
Subject: Re: [PATCH v2] create_inode: set xattrs to the root directory as well

On Sep 4, 2020, at 2:23 AM, Antoine Tenart <antoine.tenart@...tlin.com> wrote:
> 
> Hello,
> 
> Quoting Antoine Tenart (2020-07-30 09:27:50)
>> 
>> Gentle ping. What's the status of this patch?
> 
> Do anyone know if anything else is required to get this merged?

Based on other emails to the list, Ted has a backlog of patches to be
merged for ext4 and e2fsprogs, so it isn't that your patch was missed.
He will hopefully be able to start landing patches again soon.

Cheers, Andreas

>> Quoting Andreas Dilger (2020-07-17 13:17:08)
>>> 
>>>> On Jul 17, 2020, at 4:08 AM, Antoine Tenart <antoine.tenart@...tlin.com> wrote:
>>>> 
>>>> populate_fs do copy the xattrs for all files and directories, but the
>>>> root directory is skipped and as a result its extended attributes aren't
>>>> set. This is an issue when using mkfs to build a full system image that
>>>> can be used with SElinux in enforcing mode without making any runtime
>>>> fix at first boot.
>>>> 
>>>> This patch adds logic to set the root directory's extended attributes.
>>>> 
>>>> Signed-off-by: Antoine Tenart <antoine.tenart@...tlin.com>
>>> 
>>> Reviewed-by: Andreas Dilger <adilger@...ger.ca>
>>> 
>>>> ---
>>>> 
>>>> Since v1:
>>>> - Moved the set_inode_xattr logic for the root directory
>>>>   from __populate_fs to populate_fs2.
>>>> 
>>>> misc/create_inode.c | 8 ++++++++
>>>> 1 file changed, 8 insertions(+)
>>>> 
>>>> diff --git a/misc/create_inode.c b/misc/create_inode.c
>>>> index e8d1df6b55a5..fe66faf1b53d 100644
>>>> --- a/misc/create_inode.c
>>>> +++ b/misc/create_inode.c
>>>> @@ -1050,9 +1050,17 @@ errcode_t populate_fs2(ext2_filsys fs, ext2_ino_t parent_ino,
>>>>      file_info.path_max_len = 255;
>>>>      file_info.path = calloc(file_info.path_max_len, 1);
>>>> 
>>>> +     retval = set_inode_xattr(fs, root, source_dir);
>>>> +     if (retval) {
>>>> +             com_err(__func__, retval,
>>>> +                     _("while copying xattrs on root directory"));
>>>> +             goto out;
>>>> +     }
>>>> +
>>>>      retval = __populate_fs(fs, parent_ino, source_dir, root, &hdlinks,
>>>>                             &file_info, fs_callbacks);
>>>> 
>>>> +out:
>>>>      free(file_info.path);
>>>>      free(hdlinks.hdl);
>>>>      return retval;
>>>> --
>>>> 2.26.2
>>>> 
>>> 
>>> 
>>> Cheers, Andreas
>>> 
>>> 
>>> 
>>> 
>>> 
>> 
>> --
>> Antoine Ténart, Bootlin
>> Embedded Linux and Kernel engineering
>> https://bootlin.com
> 
> --
> Antoine Ténart, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com


Cheers, Andreas






Download attachment "signature.asc" of type "application/pgp-signature" (874 bytes)