Merge pull request #14 from lyft/global-sigpipe

Globally disable SIGPIPE

Author: Reflejo

Kronos.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name = 'Kronos'
-  s.version = '0.2.2'
+  s.version = '0.2.3'
   s.license = { :type => 'Apache License, Version 2.0', :file => 'LICENSE' }
   s.summary = 'Elegant NTP client in Swift'
   s.homepage = 'https://github.com/lyft/Kronos'

Sources/NTPClient.swift

@@ -145,6 +145,8 @@ final class NTPClient {
     private func sendAsyncUDPQuery(to ip: InternetAddress, port: Int, timeout: TimeInterval,
                                    completion: UnsafeMutableRawPointer) -> (CFRunLoopSource, CFSocket)?
     {
+        signal(SIGPIPE, SIG_IGN)
+
         let callback: CFSocketCallBack = { socket, callbackType, address, data, info in
             if callbackType == .writeCallBack {
                 var packet = NTPPacket()
@@ -179,9 +181,6 @@ final class NTPClient {
 
         let runLoopSource = CFSocketCreateRunLoopSource(kCFAllocatorDefault, socket, 0)
         CFRunLoopAddSource(CFRunLoopGetMain(), runLoopSource, CFRunLoopMode.commonModes)
-
-        var noSIGPIPE: UInt32 = 1
-        setsockopt(CFSocketGetNative(socket), SOL_SOCKET, SO_NOSIGPIPE, &noSIGPIPE, 4)
         CFSocketConnectToAddress(socket, ip.addressData(withPort: port), timeout)
         return (runLoopSource!, socket)
     }