diff options
| author | Christian Brauner <brauner@kernel.org> | 2025-10-07 11:32:42 +0200 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2025-10-07 13:12:46 +0200 |
| commit | a779e27f24aeb679969ddd1fdd7f636e22ddbc1e (patch) | |
| tree | a022bc55b3aa626e419785d91a5d6e95fb40a6a6 /fs/exec.c | |
| parent | 56094ad3eaa21e6621396cc33811d8f72847a834 (diff) | |
coredump: fix core_pattern input validation
In be1e0283021e ("coredump: don't pointlessly check and spew warnings")
we tried to fix input validation so it only happens during a write to
core_pattern. This would avoid needlessly logging a lot of warnings
during a read operation. However the logic accidently got inverted in
this commit. Fix it so the input validation only happens on write and is
skipped on read.
Fixes: be1e0283021e ("coredump: don't pointlessly check and spew warnings")
Fixes: 16195d2c7dd2 ("coredump: validate socket name as it is written")
Reviewed-by: Jan Kara <jack@suse.cz>
Reported-by: Yu Watanabe <watanabe.yu@gmail.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'fs/exec.c')
| -rw-r--r-- | fs/exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/exec.c b/fs/exec.c index 6b70c6726d31..4298e7e08d5d 100644 --- a/fs/exec.c +++ b/fs/exec.c @@ -2048,7 +2048,7 @@ static int proc_dointvec_minmax_coredump(const struct ctl_table *table, int writ { int error = proc_dointvec_minmax(table, write, buffer, lenp, ppos); - if (!error && !write) + if (!error && write) validate_coredump_safety(); return error; } |