pidfile_open incorrectly returns EAGAIN when pidfile is locked
Dirk Engling
erdgeist at erdgeist.org
Thu Mar 14 09:33:40 UTC 2013
On Thu, 14 Mar 2013, Pawel Jakub Dawidek wrote:
> Right. Your patch assumes EWOULDBLOCK is equal to EAGAIN, which is true
> on FreeBSD, but is not portable. Also in case pidptr is NULL you compare
> errno three times instead of just one (not a big deal of course, just
> something that could be done a bit more optimal:)).
You're right.
OTOH, wouldn't modifying pidfile_read to return EAGAIN and not EWOULDBLOCK
totally be worth saving that 2 lines in the patch? ;)
erdgeist
More information about the freebsd-current
mailing list