diff options
author | OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> | 2007-07-16 02:39:56 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-16 12:05:42 -0400 |
commit | 9aacd599342fdfc1fb9422f37e900609b7a46249 (patch) | |
tree | 45c835330122496dfd51b946890ce8580d854173 /fs/fat/inode.c | |
parent | 259902ea951008bcbd31a49f667062ff8012ef55 (diff) |
fat: gcc 4.3 warning fix
This patch fixes the following warnings.
fs/fat/dir.c: In function 'fat_parse_long':
include/linux/msdos_fs.h:294: warning: array subscript is above array bounds
include/linux/msdos_fs.h:295: warning: array subscript is above array bounds
include/linux/msdos_fs.h:295: warning: array subscript is above array bounds
The ->name is defined as "name[8], ext[3]", but fat_checksum() uses
those as name[11]. There is no actual problem, but it's not a good manner.
Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/fat/inode.c')
-rw-r--r-- | fs/fat/inode.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/fat/inode.c b/fs/fat/inode.c index 479722d89667..cfaf5877d98b 100644 --- a/fs/fat/inode.c +++ b/fs/fat/inode.c | |||
@@ -354,8 +354,7 @@ static int fat_fill_inode(struct inode *inode, struct msdos_dir_entry *de) | |||
354 | } else { /* not a directory */ | 354 | } else { /* not a directory */ |
355 | inode->i_generation |= 1; | 355 | inode->i_generation |= 1; |
356 | inode->i_mode = MSDOS_MKMODE(de->attr, | 356 | inode->i_mode = MSDOS_MKMODE(de->attr, |
357 | ((sbi->options.showexec && | 357 | ((sbi->options.showexec && !is_exec(de->name + 8)) |
358 | !is_exec(de->ext)) | ||
359 | ? S_IRUGO|S_IWUGO : S_IRWXUGO) | 358 | ? S_IRUGO|S_IWUGO : S_IRWXUGO) |
360 | & ~sbi->options.fs_fmask) | S_IFREG; | 359 | & ~sbi->options.fs_fmask) | S_IFREG; |
361 | MSDOS_I(inode)->i_start = le16_to_cpu(de->start); | 360 | MSDOS_I(inode)->i_start = le16_to_cpu(de->start); |