Theo Buehler <t...@theobuehler.org> wrote: > On Tue, Mar 02, 2021 at 02:09:37PM +0100, Claudio Jeker wrote: > > This diff just brings all err(3) calls for out of memory situations to one > > form: err(1, NULL); > > It is not very helpful to tell if malloc, strdup or asprintf failed with no > > mem. Just one common idiom. > > > > OK? > > ok. > > The https diff will again add a few more of those.
This is similar to the following pattern found throughout the tree: pflogd/privsep.c: err(1, "unveil"); I really dislike such, as obviously it should show the path. Being able to discern specific failures more accurately is obviously helpful.