On Mon, 3 May 2021 at 10:17, Theo de Raadt <dera...@openbsd.org> wrote: > So if the messages were just 'unveil %s: error' or 'unveil: %s: error' > I would be thrilled, as this allows users to realize why the program is > not working right.
Florian/Theo, Thanks for the reviews. So it sounds like err(1, "unveil: %s error", some_path); is the way to go - with the exception of unveil(NULL, NULL) as discussed since that's not ambiguous as there's no path involved. Seems like slaacd and dhcpleased should also be included in this as well for consistency's sake - I will update these diffs and continue with /usr/bin and /usr/sbin in the coming days. Cheers, Ash