Commit 62a8f46c authored by Anton Altaparmakov's avatar Anton Altaparmakov

NTFS: Change '\0' and L'\0' to simply 0 as per advice from Linus Torvalds.

Signed-off-by: default avatarAnton Altaparmakov <aia21@cantab.net>
parent 18e3056b
...@@ -30,6 +30,7 @@ ToDo/Notes: ...@@ -30,6 +30,7 @@ ToDo/Notes:
kernel. kernel.
- Get rid of the ugly transparent union in fs/ntfs/dir.c::ntfs_readdir() - Get rid of the ugly transparent union in fs/ntfs/dir.c::ntfs_readdir()
and ntfs_filldir() as per suggestion from Al Viro. and ntfs_filldir() as per suggestion from Al Viro.
- Change '\0' and L'\0' to simply 0 as per advice from Linus Torvalds.
2.1.18 - Fix scheduling latencies at mount time as well as an endianness bug. 2.1.18 - Fix scheduling latencies at mount time as well as an endianness bug.
......
...@@ -130,7 +130,7 @@ static int ntfs_init_locked_inode(struct inode *vi, ntfs_attr *na) ...@@ -130,7 +130,7 @@ static int ntfs_init_locked_inode(struct inode *vi, ntfs_attr *na)
if (!ni->name) if (!ni->name)
return -ENOMEM; return -ENOMEM;
memcpy(ni->name, na->name, i); memcpy(ni->name, na->name, i);
ni->name[i] = cpu_to_le16(L'\0'); ni->name[i] = cpu_to_le16(0);
} }
return 0; return 0;
} }
...@@ -2270,6 +2270,12 @@ int ntfs_show_options(struct seq_file *sf, struct vfsmount *mnt) ...@@ -2270,6 +2270,12 @@ int ntfs_show_options(struct seq_file *sf, struct vfsmount *mnt)
* *
* We don't support i_size changes yet. * We don't support i_size changes yet.
* *
* The kernel guarantees that @vi is a regular file (S_ISREG() is true) and
* that the change is allowed.
*
* This implies for us that @vi is a file inode rather than a directory, index,
* or attribute inode as well as that @vi is a base inode.
*
* Called with ->i_sem held. In all but one case ->i_alloc_sem is held for * Called with ->i_sem held. In all but one case ->i_alloc_sem is held for
* writing. The only case where ->i_alloc_sem is not held is * writing. The only case where ->i_alloc_sem is not held is
* mm/filemap.c::generic_file_buffered_write() where vmtruncate() is called * mm/filemap.c::generic_file_buffered_write() where vmtruncate() is called
...@@ -2281,8 +2287,8 @@ void ntfs_truncate(struct inode *vi) ...@@ -2281,8 +2287,8 @@ void ntfs_truncate(struct inode *vi)
// TODO: Implement... // TODO: Implement...
ntfs_warning(vi->i_sb, "Eeek: i_size may have changed! If you see " ntfs_warning(vi->i_sb, "Eeek: i_size may have changed! If you see "
"this right after a message from " "this right after a message from "
"ntfs_{prepare,commit}_{,nonresident_}write() then " "ntfs_prepare_{,nonresident_}write() then just ignore "
"just ignore it. Otherwise it is bad news."); "it. Otherwise it is bad news.");
// TODO: reset i_size now! // TODO: reset i_size now!
return; return;
} }
......
...@@ -145,7 +145,7 @@ static BOOL parse_options(ntfs_volume *vol, char *opt) ...@@ -145,7 +145,7 @@ static BOOL parse_options(ntfs_volume *vol, char *opt)
ntfs_debug("Entering with mount options string: %s", opt); ntfs_debug("Entering with mount options string: %s", opt);
while ((p = strsep(&opt, ","))) { while ((p = strsep(&opt, ","))) {
if ((v = strchr(p, '='))) if ((v = strchr(p, '=')))
*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("umask", fmask = dmask)
......
...@@ -276,7 +276,7 @@ int ntfs_nlstoucs(const ntfs_volume *vol, const char *ins, ...@@ -276,7 +276,7 @@ int ntfs_nlstoucs(const ntfs_volume *vol, const char *ins,
} /* else (wc_len < 0) */ } /* else (wc_len < 0) */
goto conversion_err; goto conversion_err;
} }
ucs[o] = cpu_to_le16('\0'); ucs[o] = cpu_to_le16(0);
*outs = ucs; *outs = ucs;
return o; return o;
} /* else (!ucs) */ } /* else (!ucs) */
...@@ -362,7 +362,7 @@ retry: wc = nls->uni2char(le16_to_cpu(ins[i]), ns + o, ...@@ -362,7 +362,7 @@ retry: wc = nls->uni2char(le16_to_cpu(ins[i]), ns + o,
} /* wc < 0, real error. */ } /* wc < 0, real error. */
goto conversion_err; goto conversion_err;
} }
ns[o] = '\0'; ns[o] = 0;
*outs = ns; *outs = ns;
return o; return o;
} /* else (!ins) */ } /* else (!ins) */
......
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