Re: [PATCH 1/2] fat: Clean up xlate_to_uni.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


Namjae Jeon <linkinjeon@xxxxxxxxx> writes:

>  The xlate_to_uni is called by vfat_build_slots with sbi->nls_io as the final
>  argument. nls_io can never be null at this point because the check is already being done in fat_fill_super wherein the mount fails if it is null.

Looks good to me. And now, I don't have public repo after kernel.org
break in. It would be better to pass akpm.

Acked-by: OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>

> Signed-off-by: Namjae Jeon <linkinjeon@xxxxxxxxx>
> Signed-off-by: Ravishankar N <cyberax82@xxxxxxxxx>
> ---
>  fs/fat/namei_vfat.c |   83 ++++++++++++++++++++++-----------------------------
>  1 files changed, 36 insertions(+), 47 deletions(-)
>
> diff --git a/fs/fat/namei_vfat.c b/fs/fat/namei_vfat.c
> index a81eb23..aae3b4e 100644
> --- a/fs/fat/namei_vfat.c
> +++ b/fs/fat/namei_vfat.c
> @@ -521,57 +521,46 @@ xlate_to_uni(const unsigned char *name, int len, unsigned char *outname,
>  
>  		op = &outname[*outlen * sizeof(wchar_t)];
>  	} else {
> -		if (nls) {
> -			for (i = 0, ip = name, op = outname, *outlen = 0;
> -			     i < len && *outlen <= FAT_LFN_LEN;
> -			     *outlen += 1)
> -			{
> -				if (escape && (*ip == ':')) {
> -					if (i > len - 5)
> -						return -EINVAL;
> -					ec = 0;
> -					for (k = 1; k < 5; k++) {
> -						nc = ip[k];
> -						ec <<= 4;
> -						if (nc >= '0' && nc <= '9') {
> -							ec |= nc - '0';
> -							continue;
> -						}
> -						if (nc >= 'a' && nc <= 'f') {
> -							ec |= nc - ('a' - 10);
> -							continue;
> -						}
> -						if (nc >= 'A' && nc <= 'F') {
> -							ec |= nc - ('A' - 10);
> -							continue;
> -						}
> -						return -EINVAL;
> +		for (i = 0, ip = name, op = outname, *outlen = 0;
> +			 i < len && *outlen <= FAT_LFN_LEN;
> +			 *outlen += 1) {
> +			if (escape && (*ip == ':')) {
> +				if (i > len - 5)
> +					return -EINVAL;
> +				ec = 0;
> +				for (k = 1; k < 5; k++) {
> +					nc = ip[k];
> +					ec <<= 4;
> +					if (nc >= '0' && nc <= '9') {
> +						ec |= nc - '0';
> +						continue;
>  					}
> -					*op++ = ec & 0xFF;
> -					*op++ = ec >> 8;
> -					ip += 5;
> -					i += 5;
> -				} else {
> -					if ((charlen = nls->char2uni(ip, len - i, (wchar_t *)op)) < 0)
> -						return -EINVAL;
> -					ip += charlen;
> -					i += charlen;
> -					op += 2;
> +					if (nc >= 'a' && nc <= 'f') {
> +						ec |= nc - ('a' - 10);
> +						continue;
> +					}
> +					if (nc >= 'A' && nc <= 'F') {
> +						ec |= nc - ('A' - 10);
> +						continue;
> +					}
> +					return -EINVAL;
>  				}
> +				*op++ = ec & 0xFF;
> +				*op++ = ec >> 8;
> +				ip += 5;
> +				i += 5;
> +			} else {
> +				charlen = nls->char2uni(ip, len - i,
> +									(wchar_t *)op);
> +				if (charlen < 0)
> +					return -EINVAL;
> +				ip += charlen;
> +				i += charlen;
> +				op += 2;
>  			}
> -			if (i < len)
> -				return -ENAMETOOLONG;
> -		} else {
> -			for (i = 0, ip = name, op = outname, *outlen = 0;
> -			     i < len && *outlen <= FAT_LFN_LEN;
> -			     i++, *outlen += 1)
> -			{
> -				*op++ = *ip++;
> -				*op++ = 0;
> -			}
> -			if (i < len)
> -				return -ENAMETOOLONG;
>  		}
> +		if (i < len)
> +			return -ENAMETOOLONG;
>  	}
>  
>  	*longlen = *outlen;

-- 
OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[Other Archives]     [Linux Kernel Newbies]     [Linux Driver Development]     [Fedora Kernel]     [Linux Kernel Testers]     [Linux SH]     [Linux Omap]     [Linux Kbuild]     [Linux Tape]     [Linux Input]     [Linux Kernel Janitors]     [Linux Kernel Packagers]     [Linux Doc]     [Linux Man Pages]     [Linux API]     [Linux Memory Management]     [Linux Modules]     [Linux Standards]     [Kernel Announce]     [Netdev]     [Git]     [Linux PCI]     Linux CAN Development     [Linux I2C]     [Linux RDMA]     [Linux NUMA]     [Netfilter]     [Netfilter Devel]     [SELinux]     [Bugtraq]     [FIO]     [Linux Perf Users]     [Linux Serial]     [Linux PPP]     [Linux ISDN]     [Linux Next]     [Kernel Stable Commits]     [Linux Tip Commits]     [Kernel MM Commits]     [Linux Security Module]     [Filesystem Development]     [Ext3 Filesystem]     [Linux bcache]     [Ext4 Filesystem]     [Linux BTRFS]     [Linux CEPH Filesystem]     [Linux XFS]     [XFS]     [Linux NFS]     [Linux CIFS]     [Ecryptfs]     [Linux NILFS]     [Linux Cachefs]     [Reiser FS]     [Initramfs]     [Linux FB Devel]     [Linux OpenGL]     [DRI Devel]     [Fastboot]     [Linux RT Users]     [Linux RT Stable]     [eCos]     [Corosync]     [Linux Clusters]     [LVS Devel]     [Hot Plug]     [Linux Virtualization]     [KVM]     [KVM PPC]     [KVM ia64]     [Linux Containers]     [Linux Hexagon]     [Linux Cgroups]     [Util Linux]     [Wireless]     [Linux Bluetooth]     [Bluez Devel]     [Ethernet Bridging]     [Embedded Linux]     [Barebox]     [Linux MMC]     [Linux IIO]     [Sparse]     [Smatch]     [Linux Arch]     [x86 Platform Driver]     [Linux ACPI]     [Linux IBM ACPI]     [LM Sensors]     [CPU Freq]     [Linux Power Management]     [Linmodems]     [Linux DCCP]     [Linux SCTP]     [ALSA Devel]     [Linux USB]     [Linux PA RISC]     [Linux Samsung SOC]     [MIPS Linux]     [IBM S/390 Linux]     [ARM Linux]     [ARM Kernel]     [ARM MSM]     [Tegra Devel]     [Sparc Linux]     [Linux Security]     [Linux Sound]     [Linux Media]     [Video 4 Linux]     [Linux IRDA Users]     [Linux for the blind]     [Linux RAID]     [Linux ATA RAID]     [Device Mapper]     [Linux SCSI]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Linux IDE]     [Linux SMP]     [Linux AXP]     [Linux Alpha]     [Linux M68K]     [Linux ia64]     [Linux 8086]     [Linux x86_64]     [Linux Config]     [Linux Apps]     [Linux MSDOS]     [Linux X.25]     [Linux Crypto]     [DM Crypt]     [Linux Trace Users]     [Linux Btrace]     [Linux Watchdog]     [Utrace Devel]     [Linux C Programming]     [Linux Assembly]     [Dash]     [DWARVES]     [Hail Devel]     [Linux Kernel Debugger]     [Linux gcc]     [Gcc Help]     [X.Org]     [Wine]

Add to Google Powered by Linux

[Older Kernel Discussion]     [Yosemite National Park Forum]     [Large Format Photos]     [Gimp]     [Yosemite Photos]     [Stuff]