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 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


--
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/