diff options
| author | Mateusz Guzik <mguzik@redhat.com> | 2017-10-03 12:58:14 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2017-11-05 18:58:06 -0500 |
| commit | 5297908270549b734c7c2556745e2385b6d4941d (patch) | |
| tree | 1598865e6446d2177af03d53ac2135a3a6f7e01e /fs/file.c | |
| parent | 63dcb81e5b9e1faadf4b55450141bc4446e5a3d3 (diff) | |
| download | lwn-5297908270549b734c7c2556745e2385b6d4941d.tar.gz lwn-5297908270549b734c7c2556745e2385b6d4941d.zip | |
vfs: stop clearing close on exec when closing a fd
Codepaths allocating a fd always make sure the bit is set/unset
depending on flags, thus clearing on close is redundant.
Signed-off-by: Mateusz Guzik <mguzik@redhat.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/file.c')
| -rw-r--r-- | fs/file.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/file.c b/fs/file.c index 1fc7fbbb4510..9d047bd046b0 100644 --- a/fs/file.c +++ b/fs/file.c @@ -631,7 +631,6 @@ int __close_fd(struct files_struct *files, unsigned fd) if (!file) goto out_unlock; rcu_assign_pointer(fdt->fd[fd], NULL); - __clear_close_on_exec(fd, fdt); __put_unused_fd(files, fd); spin_unlock(&files->file_lock); return filp_close(file, files); |