linux-kernel - Re: [PATCH] fat: remove unused fat_tolower() and use to

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>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <87y0l9t77q.fsf@mail.parknet.co.jp>
Date: Wed, 04 Feb 2026 14:20:57 +0900
From: OGAWA Hirofumi <hirofumi@...l.parknet.co.jp>
To: George Hu <integral@...hlinux.org>
Cc: linux-kernel@...r.kernel.org
Subject: Re: [PATCH] fat: remove unused fat_tolower() and use to_lower()
 instead

George Hu <integral@...hlinux.org> writes:

> The fat_tolower() function is redundant, as it is functionally
> equivalent to the to_lower() macro defined in include/linux/ctype.h.

No, it is not equivalent.

Thanks.

> Signed-off-by: George Hu <integral@...hlinux.org>
> ---
>  fs/fat/dir.c | 10 +++-------
>  1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/fs/fat/dir.c b/fs/fat/dir.c
> index 92b091783966..4f46d2358185 100644
> --- a/fs/fat/dir.c
> +++ b/fs/fat/dir.c
> @@ -18,6 +18,7 @@
>  #include <linux/compat.h>
>  #include <linux/uaccess.h>
>  #include <linux/iversion.h>
> +#include <linux/ctype.h>
>  #include "fat.h"
>  
>  /*
> @@ -33,11 +34,6 @@
>  #define FAT_MAX_UNI_CHARS	((MSDOS_SLOTS - 1) * 13 + 1)
>  #define FAT_MAX_UNI_SIZE	(FAT_MAX_UNI_CHARS * sizeof(wchar_t))
>  
> -static inline unsigned char fat_tolower(unsigned char c)
> -{
> -	return ((c >= 'A') && (c <= 'Z')) ? c+32 : c;
> -}
> -
>  static inline loff_t fat_make_i_pos(struct super_block *sb,
>  				    struct buffer_head *bh,
>  				    struct msdos_dir_entry *de)
> @@ -396,7 +392,7 @@ static int fat_parse_short(struct super_block *sb,
>  					de->lcase & CASE_LOWER_BASE);
>  		if (chl <= 1) {
>  			if (!isvfat)
> -				ptname[i] = nocase ? c : fat_tolower(c);
> +				ptname[i] = nocase ? c : tolower(c);
>  			i++;
>  			if (c != ' ') {
>  				name_len = i;
> @@ -433,7 +429,7 @@ static int fat_parse_short(struct super_block *sb,
>  		if (chl <= 1) {
>  			k++;
>  			if (!isvfat)
> -				ptname[i] = nocase ? c : fat_tolower(c);
> +				ptname[i] = nocase ? c : tolower(c);
>  			i++;
>  			if (c != ' ') {
>  				name_len = i;

-- 
OGAWA Hirofumi <hirofumi@...l.parknet.co.jp>