Remi Locherer <remi.loche...@relo.ch> wrote: > On Tue, Oct 30, 2018 at 10:54:10AM -0600, Theo de Raadt wrote: > > Remi Locherer <remi.loche...@relo.ch> wrote: > > > > > On Tue, Oct 30, 2018 at 03:20:35PM +0000, Ricardo Mestre wrote: > > > > Hi, > > > > > > > > After all files are opened ripd(8) can have the fs access disabled just > > > > before > > > > each process main loop. Its 2 childs already run under chroot, but > > > > since they > > > > are still not pledged at least they have no way to read/write/create > > > > files within > > > > the chroot. No loads or reloads of the config file happen through any > > > > signal, > > > > nor can we do it via ripctl(8). > > > > > > > > I was able to run a simple daemon with the example file. Comments? OK? > > > > > > control_cleanup() unlinks the control socket on exit. I think you should > > > either unveil(conf->csock, "c") or remove control_cleanup(). > > > > I don't understand this latter comment, let me ask. > > > > You think it is smart to leave these sockets lying around? > > > > I suspect there are a few oddball programs where it makes senes, but as > > a general rule I think it is a bad idea; as stated in other threads it > > means control programs and restart sequences have a bunch more oddball > > behaviours which will be inconsistant. > > > > I prefer if sockets get removed on exit. But I was not sure if this is > just my personal taste.
I didn't speak about taste, I spoke about behaviour relative to the control program. Why do you propose potentially changing behaviour without checking first?