Messages in this thread Patch in this message | | | Date | Tue, 9 Nov 2004 19:22:24 +0100 | From | Rene Scharfe <> | Subject | Re: [PATCH 1/4] Move check for invalid chars to vfat_valid_longname() |
| |
On Wed, Nov 10, 2004 at 02:25:28AM +0900, OGAWA Hirofumi wrote: > René Scharfe <rene.scharfe@lsrfire.ath.cx> writes: > > > We want to make sure filenames don't contain '*', '?' etc. > > No. For example, Shift-JIS has 0x815c. It's contains the '\' (0x5c), > but the 0x815c is a valid char for fatfs.
Oops -- of course.
But doesn't imply this we can't do any of our checks on the VFS string? A dot (0x2E) at the end of a filename could be the half of some other character in some encoding, right? And the same could be said about the checks in vfat_valid_longname(), no? Do we need to convert them to Unicode?
The patch you asked for converting IS_BADCHAR to an inline function follows. I rolled it together with the other conversions from patch 3. Applies directly on top of 2.6.10-rc1-bk18.
Thanks, René
--- linux-2.6.10-rc1-bk18/fs/vfat/namei.c.orig 2004-10-18 23:54:37.000000000 +0200 +++ linux-2.6.10-rc1-bk18/fs/vfat/namei.c 2004-11-09 18:56:48.000000000 +0100 @@ -158,33 +158,34 @@ static int vfat_cmp(struct dentry *dentr /* Characters that are undesirable in an MS-DOS file name */ -static wchar_t bad_chars[] = { - /* `*' `?' `<' `>' `|' `"' `:' `/' */ - 0x002A, 0x003F, 0x003C, 0x003E, 0x007C, 0x0022, 0x003A, 0x002F, - /* `\' */ - 0x005C, 0, -}; -#define IS_BADCHAR(uni) (vfat_unistrchr(bad_chars, (uni)) != NULL) - -static wchar_t replace_chars[] = { - /* `[' `]' `;' `,' `+' `=' */ - 0x005B, 0x005D, 0x003B, 0x002C, 0x002B, 0x003D, 0, -}; -#define IS_REPLACECHAR(uni) (vfat_unistrchr(replace_chars, (uni)) != NULL) - -static wchar_t skip_chars[] = { - /* `.' ` ' */ - 0x002E, 0x0020, 0, -}; -#define IS_SKIPCHAR(uni) \ - ((wchar_t)(uni) == skip_chars[0] || (wchar_t)(uni) == skip_chars[1]) +static inline wchar_t vfat_bad_char(wchar_t w) +{ + return (w < 0x0020) + || (w == 0x002A) /* * */ + || (w == 0x003F) /* ? */ + || (w == 0x003C) /* < */ + || (w == 0x003E) /* > */ + || (w == 0x007C) /* | */ + || (w == 0x0022) /* " */ + || (w == 0x003A) /* : */ + || (w == 0x002F) /* / */ + || (w == 0x005C); /* \ */ +} + +static inline wchar_t vfat_replace_char(wchar_t w) +{ + return (w == 0x005B) /* [ */ + || (w == 0x005D) /* ] */ + || (w == 0x003B) /* ; */ + || (w == 0x002C) /* , */ + || (w == 0x002B) /* + */ + || (w == 0x003D); /* = */ +} -static inline wchar_t *vfat_unistrchr(const wchar_t *s, const wchar_t c) +static inline wchar_t vfat_skip_char(wchar_t w) { - for(; *s != c; ++s) - if (*s == 0) - return NULL; - return (wchar_t *) s; + return (w == 0x002E) /* . */ + || (w == 0x0020); /* <space> */ } static inline int vfat_is_used_badchars(const wchar_t *s, int len) @@ -192,7 +193,7 @@ static inline int vfat_is_used_badchars( int i; for (i = 0; i < len; i++) - if (s[i] < 0x0020 || IS_BADCHAR(s[i])) + if (vfat_bad_char(s[i])) return -EINVAL; return 0; } @@ -291,11 +292,11 @@ static inline int to_shortname_char(stru { int len; - if (IS_SKIPCHAR(*src)) { + if (vfat_skip_char(*src)) { info->valid = 0; return 0; } - if (IS_REPLACECHAR(*src)) { + if (vfat_replace_char(*src)) { info->valid = 0; buf[0] = '_'; return 1; @@ -375,7 +376,7 @@ static int vfat_create_shortname(struct */ name_start = &uname[0]; while (name_start < ext_start) { - if (!IS_SKIPCHAR(*name_start)) + if (!vfat_skip_char(*name_start)) break; name_start++; } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |