Commit 5d46770f authored by Anton Altaparmakov's avatar Anton Altaparmakov

NTFS: Change the mount options {u,f,d}mask to always parse the number as

      an octal number to conform to how chmod(1) works, too.  Thanks to
      Giuseppe Bilotta and Horst von Brand for pointing out the errors of
      my ways.
Signed-off-by: default avatarAnton Altaparmakov <aia21@cantab.net>
parent c93a7771
...@@ -90,7 +90,11 @@ ToDo/Notes: ...@@ -90,7 +90,11 @@ ToDo/Notes:
in the first buffer head instead of a driver global spin lock to in the first buffer head instead of a driver global spin lock to
improve scalability. improve scalability.
- Minor fix to error handling and error message display in - Minor fix to error handling and error message display in
fs/ntfs/aops.c::ntfs_prepare_nonresident_write(). fs/ntfs/aops.c::ntfs_prepare_nonresident_write().
- Change the mount options {u,f,d}mask to always parse the number as
an octal number to conform to how chmod(1) works, too. Thanks to
Giuseppe Bilotta and Horst von Brand for pointing out the errors of
my ways.
2.1.23 - Implement extension of resident files and make writing safe as well as 2.1.23 - Implement extension of resident files and make writing safe as well as
many bug fixes, cleanups, and enhancements... many bug fixes, cleanups, and enhancements...
......
...@@ -126,6 +126,14 @@ static BOOL parse_options(ntfs_volume *vol, char *opt) ...@@ -126,6 +126,14 @@ static BOOL parse_options(ntfs_volume *vol, char *opt)
if (*v) \ if (*v) \
goto needs_val; \ goto needs_val; \
} }
#define NTFS_GETOPT_OCTAL(option, variable) \
if (!strcmp(p, option)) { \
if (!v || !*v) \
goto needs_arg; \
variable = simple_strtoul(ov = v, &v, 8); \
if (*v) \
goto needs_val; \
}
#define NTFS_GETOPT_BOOL(option, variable) \ #define NTFS_GETOPT_BOOL(option, variable) \
if (!strcmp(p, option)) { \ if (!strcmp(p, option)) { \
BOOL val; \ BOOL val; \
...@@ -157,9 +165,9 @@ static BOOL parse_options(ntfs_volume *vol, char *opt) ...@@ -157,9 +165,9 @@ static BOOL parse_options(ntfs_volume *vol, char *opt)
*v++ = 0; *v++ = 0;
NTFS_GETOPT("uid", uid) NTFS_GETOPT("uid", uid)
else NTFS_GETOPT("gid", gid) else NTFS_GETOPT("gid", gid)
else NTFS_GETOPT("umask", fmask = dmask) else NTFS_GETOPT_OCTAL("umask", fmask = dmask)
else NTFS_GETOPT("fmask", fmask) else NTFS_GETOPT_OCTAL("fmask", fmask)
else NTFS_GETOPT("dmask", dmask) else NTFS_GETOPT_OCTAL("dmask", dmask)
else NTFS_GETOPT("mft_zone_multiplier", mft_zone_multiplier) else NTFS_GETOPT("mft_zone_multiplier", mft_zone_multiplier)
else NTFS_GETOPT_WITH_DEFAULT("sloppy", sloppy, TRUE) else NTFS_GETOPT_WITH_DEFAULT("sloppy", sloppy, TRUE)
else NTFS_GETOPT_BOOL("show_sys_files", show_sys_files) else NTFS_GETOPT_BOOL("show_sys_files", show_sys_files)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment