Re: [PATCH] error-injection: Add prompt for function error injection

From: Steven Rostedt
Date: Mon Nov 21 2022 - 19:40:34 EST


On Mon, 21 Nov 2022 19:24:23 -0500
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> On Mon, 21 Nov 2022 15:36:08 -0800
> Alexei Starovoitov <alexei.starovoitov@xxxxxxxxx> wrote:
>
> > The commit log is bogus and the lack of understanding what
> > bpf and error injection hooks are used for expressed
> > in this thread is quite sad.
> > Disabling error injection makes the system _less_ secure.
>
> Please specify.
>
> As Masami replied, you are abusing this feature for some arcane way to do
> security. It's "error injection" how does enabling this improve security???
>

If you want to add BPF programs to determine who or what can access various
functions, then please work with the security folks and hook into their
infrastructure. Please do not make some home grown operations on top of an
interface that was not created for this purpose. That can only lead to
unexpected consequences.

-- Steve