Re: [chrony-dev] [GIT] chrony/chrony.git branch, master, updated. 2.2-48-gfe50212

[ Thread Index | Date Index | More chrony.tuxfamily.org/chrony-dev Archives ]


> On 9/12/2015, at 7:57 AM, Bryan Christianson <bryan@xxxxxxxxxxxxx> wrote:
> 
> 
>> On 9/12/2015, at 6:08 AM, Miroslav Lichvar <mlichvar@xxxxxxxxxx> wrote:
>> 
>> On Tue, Dec 08, 2015 at 06:04:00PM +0100, git@xxxxxxxxxxxxx wrote:
>>> Summary of changes:
>>> configure    |   18 ++++++++----
>>> main.c       |    5 +++-
>>> privops.c    |   90 +++++++++++++++++++++++++++++++++++++++++++++++-----------
>>> privops.h    |    5 ++++
>>> sys_linux.c  |    2 +-
>>> sys_macosx.c |    5 +---
>>> sys_netbsd.c |    3 ++
>>> 7 files changed, 99 insertions(+), 29 deletions(-)
>> 
>> Bryan, can you please check if it still works for you on Mac OS X?
> 
>  if (
> #ifdef SOCK_SEQPACKET
>      socketpair(AF_UNIX, SOCK_SEQPACKET, 0, sock_pair) &&
> #endif
>      socketpair(AF_UNIX, SOCK_DGRAM, 0, sock_pair))
>    LOG_FATAL(LOGF_PrivOps, "socketpair() failed : %s", strerror(errno));
> 
> SOCK_SEQPACKET is available on OS X. I don't really know the semantics of this type of socket but the above code looks like its leaking 2 descriptors

duh!! Figured it out - sorry about the noise.

Bryan Christianson
bryan@xxxxxxxxxxxxx




--
To unsubscribe email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "unsubscribe" in the subject.
For help email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "help" in the subject.
Trouble?  Email listmaster@xxxxxxxxxxxxxxxxxxxx.


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/