fat: fix parse_options()

Previous thread: reiserfs: remove double definitions of xattr macros by Linux Kernel Mailing List on Friday, July 25, 2008 - 12:04 pm. (1 message)

Next thread: fat: fix VFAT_IOCTL_READDIR_xxx and cleanup for userland by Linux Kernel Mailing List on Friday, July 25, 2008 - 12:04 pm. (1 message)
From: Linux Kernel Mailing List
Date: Friday, July 25, 2008 - 12:04 pm

Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=8d44d9...
Commit:     8d44d9741f6808c107a144f469fb89e6fe7c55e3
Parent:     3264d4ded4d916d294d776b77b72d477c63ac3be
Author:     OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
AuthorDate: Fri Jul 25 01:46:41 2008 -0700
Committer:  Linus Torvalds <torvalds@linux-foundation.org>
CommitDate: Fri Jul 25 10:53:34 2008 -0700

    fat: fix parse_options()
    
    Current parse_options() exits too early.  We need to run the code of
    bottom in this function even if users doesn't specify options.
    
    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>
---
 fs/fat/inode.c |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/fs/fat/inode.c b/fs/fat/inode.c
index 46a4508..60deb5f 100644
--- a/fs/fat/inode.c
+++ b/fs/fat/inode.c
@@ -950,7 +950,7 @@ static int parse_options(char *options, int is_vfat, int silent, int *debug,
 	*debug = 0;
 
 	if (!options)
-		return 0;
+		goto out;
 
 	while ((p = strsep(&options, ",")) != NULL) {
 		int token;
@@ -1104,10 +1104,13 @@ static int parse_options(char *options, int is_vfat, int silent, int *debug,
 			return -EINVAL;
 		}
 	}
+
+out:
 	/* UTF-8 doesn't provide FAT semantics */
 	if (!strcmp(opts->iocharset, "utf8")) {
 		printk(KERN_ERR "FAT: utf8 is not a recommended IO charset"
-		       " for FAT filesystems, filesystem will be case sensitive!\n");
+		       " for FAT filesystems, filesystem will be "
+		       "case sensitive!\n");
 	}
 
 	/* If user doesn't specify allow_utime, it's initialized from dmask. */
--

Previous thread: reiserfs: remove double definitions of xattr macros by Linux Kernel Mailing List on Friday, July 25, 2008 - 12:04 pm. (1 message)

Next thread: fat: fix VFAT_IOCTL_READDIR_xxx and cleanup for userland by Linux Kernel Mailing List on Friday, July 25, 2008 - 12:04 pm. (1 message)