diff options
| author |   David Howells <dhowells@redhat.com> | 2015-03-17 22:16:40 +0000 |
|---|---|---|
| committer |   Al Viro <viro@zeniv.linux.org.uk> | 2015-04-15 15:05:30 -0400 |
| commit | 4bbcbd3b11dbc676a272be508e47d1c4a5056349 (patch) | |
| tree | 2672c860238fac8efb1cea46a74cdbb99284e620 /fs | |
| parent | VFS: Fix up debugfs to use d_is_dir() in place of S_ISDIR() (diff) | |
| download | linux-dev-4bbcbd3b11dbc676a272be508e47d1c4a5056349.tar.xz linux-dev-4bbcbd3b11dbc676a272be508e47d1c4a5056349.zip | |
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')
| -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: error = -ENOTDIR; if ((nd->flags & LOOKUP_DIRECTORY) && !d_can_lookup(nd->path.dentry)) goto out; - if (!S_ISREG(nd->inode->i_mode)) + if (!d_is_reg(nd->path.dentry)) will_truncate = false; if (will_truncate) { |