On Sun, 4 Aug 2024 at 11:53, Oleg Nesterov <oleg@xxxxxxxxxx> wrote:
>
> Apart from SIGKILL, the dumper already has the full control.
What do you mean? It's a regular usermodehelper. It gets the dump data
as input. That's all the control it has.
> And note that the dumper can already use ptrace.
.. with the normal ptrace() rules, yes.
You realize that some setups literally disable ptrace() system calls,
right? Which your patch now effectively sidesteps.
THAT is why I don't like it. ptrace() is *dangerous*. It is very
typically one of the things that people limit for various reasons.
Just adding some implicit tracing willy-nilly needs to be something
people really worry about.
Linus
