diff options
author | Christian Brauner <brauner@kernel.org> | 2025-04-14 15:55:05 +0200 |
---|---|---|
committer | Christian Brauner <brauner@kernel.org> | 2025-05-02 14:28:45 +0200 |
commit | c57f07b235871c9e5bffaccd458dca2d9a62b164 (patch) | |
tree | cad23436a546b8fd565b6ca260769dce18813bfe | |
parent | e1b477c21300a928a7765a03cb96ca36c4dcf5ed (diff) |
pidfs: move O_RDWR into pidfs_alloc_file()
Since all pidfds must be O_RDWR currently enfore that directly in the
file allocation function itself instead of letting callers specify it.
Link: https://lore.kernel.org/20250414-work-coredump-v2-1-685bf231f828@kernel.org
Tested-by: Luca Boccassi <luca.boccassi@gmail.com>
Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
-rw-r--r-- | fs/pidfs.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/pidfs.c b/fs/pidfs.c index 9d993ecadad7..3b39e471840b 100644 --- a/fs/pidfs.c +++ b/fs/pidfs.c @@ -892,6 +892,7 @@ struct file *pidfs_alloc_file(struct pid *pid, unsigned int flags) return ERR_PTR(-ESRCH); flags &= ~PIDFD_STALE; + flags |= O_RDWR; pidfd_file = dentry_open(&path, flags, current_cred()); /* Raise PIDFD_THREAD explicitly as do_dentry_open() strips it. */ if (!IS_ERR(pidfd_file)) |