linux-kernel - RE: [PATCH v1] exfat: remove the code that sets FileAttributes when renaming

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Wed, 14 Sep 2022 17:48:10 +0900
From:   "Sungjong Seo" <sj1557.seo@...sung.com>
To:     "'Rover'" <739817562@...com>,
        "'linkinjeon'" <linkinjeon@...nel.org>
Cc:     "'linux-kernel'" <linux-kernel@...r.kernel.org>,
        "'linux-fsdevel'" <linux-fsdevel@...r.kernel.org>,
        <sj1557.seo@...sung.com>
Subject: RE: [PATCH v1] exfat: remove the code that sets FileAttributes when
 renaming

Hi, Rover,

This patch seems to violate the exFAT specification below.
Please refer to the description for ATTR_ARCHIVE in FAT32 Spec.

* Archive
This field is mandatory and conforms to the MS-DOS definition.

* ATTR_ARCHIVE
This attribute supports backup utilities. This bit is set by the FAT file
system driver when a file is created, renamed, or written to. Backup
utilities may use this attribute to indicate which files on the volume
have been modified since the last time that a backup was performed.

Thanks.
B.R.

Sungjong Seo

> When renaming, FileAttributes remain unchanged, do not need to be
> set, so the code that sets FileAttributes is unneeded, remove it.
> 
> Signed-off-by: rover &lt;739817562@...com&gt;
> ---
>  fs/exfat/namei.c | 12 ------------
>  1 file changed, 12 deletions(-)
> 
> diff --git a/fs/exfat/namei.c b/fs/exfat/namei.c
> index b617bebc3d0f..5ffaf553155e 100644
> --- a/fs/exfat/namei.c
> +++ b/fs/exfat/namei.c
> @@ -1031,10 +1031,6 @@ static int exfat_rename_file(struct inode *inode,
> struct exfat_chain *p_dir,
>  			return -EIO;
> 
>  		*epnew = *epold;
> -		if (exfat_get_entry_type(epnew) == TYPE_FILE) {
> -			epnew-&gt;dentry.file.attr |=
> cpu_to_le16(ATTR_ARCHIVE);
> -			ei-&gt;attr |= ATTR_ARCHIVE;
> -		}
>  		exfat_update_bh(new_bh, sync);
>  		brelse(old_bh);
>  		brelse(new_bh);
> @@ -1063,10 +1059,6 @@ static int exfat_rename_file(struct inode *inode,
> struct exfat_chain *p_dir,
>  		ei-&gt;dir = *p_dir;
>  		ei-&gt;entry = newentry;
>  	} else {
> -		if (exfat_get_entry_type(epold) == TYPE_FILE) {
> -			epold-&gt;dentry.file.attr |=
> cpu_to_le16(ATTR_ARCHIVE);
> -			ei-&gt;attr |= ATTR_ARCHIVE;
> -		}
>  		exfat_update_bh(old_bh, sync);
>  		brelse(old_bh);
>  		ret = exfat_init_ext_entry(inode, p_dir, oldentry,
> @@ -1112,10 +1104,6 @@ static int exfat_move_file(struct inode *inode,
> struct exfat_chain *p_olddir,
>  		return -EIO;
> 
>  	*epnew = *epmov;
> -	if (exfat_get_entry_type(epnew) == TYPE_FILE) {
> -		epnew-&gt;dentry.file.attr |= cpu_to_le16(ATTR_ARCHIVE);
> -		ei-&gt;attr |= ATTR_ARCHIVE;
> -	}
>  	exfat_update_bh(new_bh, IS_DIRSYNC(inode));
>  	brelse(mov_bh);
>  	brelse(new_bh);
> --
> 2.25.1