diff options
| author | Wang Jinchao <[email protected]> | 2024-01-31 10:54:41 +0800 |
|---|---|---|
| committer | Christian Brauner <[email protected]> | 2024-02-02 13:12:28 +0100 |
| commit | b639585e71e63008373d3a9fd060b87315fe7ea8 (patch) | |
| tree | ce3fab4d9b0ad9c8330ed42f030f4112e46cd428 | |
| parent | 6613476e225e090cc9aad49be7fa504e290dd33d (diff) | |
fork: Using clone_flags for legacy clone check
In the current implementation of clone(), there is a line that
initializes `u64 clone_flags = args->flags` at the top.
This means that there is no longer a need to use args->flags
for the legacy clone check.
Signed-off-by: Wang Jinchao <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Christian Brauner <[email protected]>
| -rw-r--r-- | kernel/fork.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/fork.c b/kernel/fork.c index 47ff3b35352e..95647c66309f 100644 --- a/kernel/fork.c +++ b/kernel/fork.c @@ -2875,8 +2875,8 @@ pid_t kernel_clone(struct kernel_clone_args *args) * here has the advantage that we don't need to have a separate helper * to check for legacy clone(). */ - if ((args->flags & CLONE_PIDFD) && - (args->flags & CLONE_PARENT_SETTID) && + if ((clone_flags & CLONE_PIDFD) && + (clone_flags & CLONE_PARENT_SETTID) && (args->pidfd == args->parent_tid)) return -EINVAL; |