diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2015-12-06 12:33:02 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2015-12-06 12:33:02 -0500 |
commit | 2788cc47f4593cca2c3c73c7bb82cd32b88c8ef7 (patch) | |
tree | 09b9de3f1e714873e1cfb947257c2de5a1040f5f | |
parent | 0f7ff2dabbc95ed7a8019d142274f0c7e083577d (diff) |
Don't reset ->total_link_count on nested calls of vfs_path_lookup()
we already zero it on outermost set_nameidata(), so initialization in
path_init() is pointless and wrong. The same DoS exists on pre-4.2
kernels, but there a slightly different fix will be needed.
Cc: stable@vger.kernel.org # v4.2
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | fs/namei.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/namei.c b/fs/namei.c index d84d7c7515fc..0c3974cd3ecd 100644 --- a/fs/namei.c +++ b/fs/namei.c | |||
@@ -1996,7 +1996,6 @@ static const char *path_init(struct nameidata *nd, unsigned flags) | |||
1996 | nd->last_type = LAST_ROOT; /* if there are only slashes... */ | 1996 | nd->last_type = LAST_ROOT; /* if there are only slashes... */ |
1997 | nd->flags = flags | LOOKUP_JUMPED | LOOKUP_PARENT; | 1997 | nd->flags = flags | LOOKUP_JUMPED | LOOKUP_PARENT; |
1998 | nd->depth = 0; | 1998 | nd->depth = 0; |
1999 | nd->total_link_count = 0; | ||
2000 | if (flags & LOOKUP_ROOT) { | 1999 | if (flags & LOOKUP_ROOT) { |
2001 | struct dentry *root = nd->root.dentry; | 2000 | struct dentry *root = nd->root.dentry; |
2002 | struct inode *inode = root->d_inode; | 2001 | struct inode *inode = root->d_inode; |