diff options
author | David Howells <dhowells@redhat.com> | 2015-03-17 18:16:40 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2015-04-15 15:05:30 -0400 |
commit | 4bbcbd3b11dbc676a272be508e47d1c4a5056349 (patch) | |
tree | 2672c860238fac8efb1cea46a74cdbb99284e620 /fs/namei.c | |
parent | 7ceab50c0be56cf1bbaf2b3dd1c6cda80e5335fb (diff) |
VFS: Make pathwalk use d_is_reg() rather than S_ISREG()
Make pathwalk use d_is_reg() rather than S_ISREG() to determine whether to
honour O_TRUNC. Since this occurs after complete_walk(), the dentry type
field cannot change and the inode pointer cannot change as we hold a ref on
the dentry, so this should be safe.
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/namei.c')
-rw-r--r-- | fs/namei.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/namei.c b/fs/namei.c index 5a9291c31881..ffab2e06e147 100644 --- a/fs/namei.c +++ b/fs/namei.c | |||
@@ -3077,7 +3077,7 @@ finish_open: | |||
3077 | error = -ENOTDIR; | 3077 | error = -ENOTDIR; |
3078 | if ((nd->flags & LOOKUP_DIRECTORY) && !d_can_lookup(nd->path.dentry)) | 3078 | if ((nd->flags & LOOKUP_DIRECTORY) && !d_can_lookup(nd->path.dentry)) |
3079 | goto out; | 3079 | goto out; |
3080 | if (!S_ISREG(nd->inode->i_mode)) | 3080 | if (!d_is_reg(nd->path.dentry)) |
3081 | will_truncate = false; | 3081 | will_truncate = false; |
3082 | 3082 | ||
3083 | if (will_truncate) { | 3083 | if (will_truncate) { |