Re: pidfile_open incorrectly returns EAGAIN when pidfile is locked

From: Dirk Engling <erdgeist_at_erdgeist.org>
Date: Thu, 14 Mar 2013 10:33:35 +0100 (CET)
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
Received on Thu Mar 14 2013 - 08:33:39 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:35 UTC