debian-packages/vibe-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge pull request #75 from vibe-d/fix_connect_tcp_cancellation

Browse source 
Fix cancellation of connectTCP in case of early errors.
This commit is contained in:
Sönke Ludwig 2018-03-18 21:29:20 +01:00 committed by GitHub
parent 88bcae036c e2eef799f8
commit 267e3c352d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
source/vibe/core/net.d
View file

@ -39,6 +39,7 @@ NetworkAddress resolveHost(string host, ushort address_family, bool use_dns = tr
else import core.sys.posix.netinet.in_ : sockaddr_in, sockaddr_in6;
enforce(host.length > 0, "Host name must not be empty.");
// FIXME: this check needs to be more specific to not disallow valid DNS names
if (host[0] == ':' || host[0] >= '0' && host[0] <= '9') {
auto addr = parseAddress(host);
enforce(address_family == AddressFamily.UNSPEC || addr.addressFamily == address_family);
@ -135,6 +136,7 @@ TCPListener listenTCP(ushort port, void delegate(TCPConnection) connection_callb
catch (Exception e) {
logError("Handling of connection failed: %s", e.msg);
conn.close();
logDebug("Full error: %s", e);
}
}, address, options);
}
@ -204,6 +206,7 @@ TCPConnection connectTCP(NetworkAddress addr, NetworkAddress bind_address = anyA
cb => eventDriver.sockets.connectStream(uaddr, baddr, cb),
(cb, sock_fd) {
cancelled = true;
if (sock_fd != SocketFD.invalid)
eventDriver.sockets.cancelConnectStream(sock_fd);
},
(fd, st) { sock = fd; status = st; }