[LTP] [PATCH v2 1/2] read_all: move blacklist to source
Jan Stancek
jstancek@redhat.com
Wed Nov 6 16:28:37 CET 2019
----- Original Message -----
> > +static char *blacklist[] = {
> > + NULL, /* reserved for -e parameter */
> > + "/sys/power/wakeup_count",
> > +};
>
> The problem with this is that it is only required if we are running as a
> privileged user. If -p is specified then it would be a bug if nobody can
> read from any of these files.
>
> So I guess we could disable the builtin blacklist if drop_privs (switch
> to nobody) is specified
Good point. I just saw your reply that you plan to send a patch, thank you.
> and run this test twice on /sys with and without
> -p.
greg-kh wasn't very happy to hear about privileged runs in the other thread.
He was suggesting whitelist approach, but I don't know how we would keep it
up to date, deal with different configs, etc.
More information about the ltp
mailing list