Re: patch: fix top

2015-11-05 Thread Theo Buehler
On Thu, Nov 05, 2015 at 10:42:54AM +0100, Marc Espie wrote: > - if (pledge("stdio rpath getpw tty id ps vminfo", NULL) == -1) > + if (pledge("stdio rpath getpw tty proc id ps vminfo", NULL) == -1) > err(1, "pledge"); I think both "proc" and "id" are too much: "id" was added

Re: patch: fix top

2015-11-05 Thread Theo Buehler
On Thu, Nov 05, 2015 at 11:33:56AM +0100, Marc Espie wrote: > On Thu, Nov 05, 2015 at 10:54:32AM +0100, Theo Buehler wrote: > > On Thu, Nov 05, 2015 at 10:42:54AM +0100, Marc Espie wrote: > > > - if (pledge("stdio rpath getpw tty id ps vminfo", NULL) == -1) > > > + if (pledge("stdio rpath getpw

Re: patch: fix top

2015-11-05 Thread Marc Espie
On Thu, Nov 05, 2015 at 10:54:32AM +0100, Theo Buehler wrote: > On Thu, Nov 05, 2015 at 10:42:54AM +0100, Marc Espie wrote: > > - if (pledge("stdio rpath getpw tty id ps vminfo", NULL) == -1) > > + if (pledge("stdio rpath getpw tty proc id ps vminfo", NULL) == -1) > > err(1,

Re: patch: fix top

2015-11-05 Thread Michael McConville
Marc Espie wrote: > On Thu, Nov 05, 2015 at 11:52:32AM +0100, Theo Buehler wrote: > > On Thu, Nov 05, 2015 at 11:33:56AM +0100, Marc Espie wrote: > > > On Thu, Nov 05, 2015 at 10:54:32AM +0100, Theo Buehler wrote: > > > > On Thu, Nov 05, 2015 at 10:42:54AM +0100, Marc Espie wrote: > > > > > -

Re: patch: fix top

2015-11-05 Thread Theo Buehler
On Thu, Nov 05, 2015 at 11:52:34AM -0500, Michael McConville wrote: > > It's not documented so it doesn't exist for me. :P > > > > (hint hint) > > Seconded. Here is an update for some missing syscalls for "stdio", "proc" and "id": Index: lib/libc/sys/pledge.2

Re: patch: fix top

2015-11-05 Thread Michael McConville
Michael McConville wrote: > > It's not documented so it doesn't exist for me. :P > > > > (hint hint) > > Seconded. I thought thought that this was referring to the undocumented top option. Disregard. :)

Re: patch: fix top

2015-11-05 Thread Marc Espie
On Thu, Nov 05, 2015 at 06:23:52PM +0100, Theo Buehler wrote: > On Thu, Nov 05, 2015 at 11:52:34AM -0500, Michael McConville wrote: > > > It's not documented so it doesn't exist for me. :P > > > > > > (hint hint) > > > > Seconded. > > Here is an update for some missing syscalls for > "stdio",

Re: patch: fix top

2015-11-05 Thread Marc Espie
On Thu, Nov 05, 2015 at 11:52:32AM +0100, Theo Buehler wrote: > On Thu, Nov 05, 2015 at 11:33:56AM +0100, Marc Espie wrote: > > On Thu, Nov 05, 2015 at 10:54:32AM +0100, Theo Buehler wrote: > > > On Thu, Nov 05, 2015 at 10:42:54AM +0100, Marc Espie wrote: > > > > - if (pledge("stdio rpath