[LTP] [PATCH v2 1/2] read_all: move blacklist to source

Cyril Hrubis chrubis@suse.cz
Wed Nov 6 14:27:52 CET 2019


Hi!
> > +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.

Good point.

> So I guess we could disable the builtin blacklist if drop_privs (switch
> to nobody) is specified and run this test twice on /sys with and without

Sounds reasonable, will you send a patch?

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list