Merge pull request #44 from vibe-d/cloexec

Set O_CLOEXEC on socket descriptors
merged-on-behalf-of: Sönke Ludwig <s-ludwig@users.noreply.github.com>
This commit is contained in:
The Dlang Bot 2017-12-18 17:35:18 +01:00 committed by GitHub
commit 94028e56dd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 40 additions and 8 deletions

View file

@ -20,11 +20,22 @@ version (Posix) {
version (linux) enum SO_REUSEPORT = 15; version (linux) enum SO_REUSEPORT = 15;
else enum SO_REUSEPORT = 0x200; else enum SO_REUSEPORT = 0x200;
static if (!is(typeof(O_CLOEXEC)))
{
version (linux) enum O_CLOEXEC = 0x80000;
else version (FreeBSD) enum O_CLOEXEC = 0x100000;
else version (NetBSD) enum O_CLOEXEC = 0x400000;
else version (OpenBSD) enum O_CLOEXEC = 0x10000;
else version (OSX) enum O_CLOEXEC = 0x1000000;
}
} }
version (linux) { version (linux) {
extern (C) int accept4(int sockfd, sockaddr *addr, socklen_t *addrlen, int flags) nothrow @nogc; extern (C) int accept4(int sockfd, sockaddr *addr, socklen_t *addrlen, int flags) nothrow @nogc;
static if (!is(typeof(SOCK_NONBLOCK))) static if (!is(typeof(SOCK_NONBLOCK)))
enum SOCK_NONBLOCK = 0x800; enum SOCK_NONBLOCK = 0x800;
static if (!is(typeof(SOCK_CLOEXEC)))
enum SOCK_CLOEXEC = 0x80000;
static if (__VERSION__ < 2077) static if (__VERSION__ < 2077)
{ {
@ -230,12 +241,12 @@ final class PosixEventDriverSockets(Loop : PosixEventLoop) : EventDriverSockets
sockaddr_storage addr; sockaddr_storage addr;
socklen_t addr_len = addr.sizeof; socklen_t addr_len = addr.sizeof;
version (linux) { version (linux) {
() @trusted { sockfd = accept4(cast(sock_t)listenfd, () @trusted { return cast(sockaddr*)&addr; } (), &addr_len, SOCK_NONBLOCK); } (); () @trusted { sockfd = accept4(cast(sock_t)listenfd, () @trusted { return cast(sockaddr*)&addr; } (), &addr_len, SOCK_NONBLOCK | SOCK_CLOEXEC); } ();
if (sockfd == -1) return; if (sockfd == -1) return;
} else { } else {
() @trusted { sockfd = accept(cast(sock_t)listenfd, () @trusted { return cast(sockaddr*)&addr; } (), &addr_len); } (); () @trusted { sockfd = accept(cast(sock_t)listenfd, () @trusted { return cast(sockaddr*)&addr; } (), &addr_len); } ();
if (sockfd == -1) return; if (sockfd == -1) return;
setSocketNonBlocking(cast(SocketFD)sockfd); setSocketNonBlocking(cast(SocketFD)sockfd, true);
} }
auto fd = cast(StreamSocketFD)sockfd; auto fd = cast(StreamSocketFD)sockfd;
m_loop.initFD(fd, FDFlags.none); m_loop.initFD(fd, FDFlags.none);
@ -850,12 +861,12 @@ final class PosixEventDriverSockets(Loop : PosixEventLoop) : EventDriverSockets
{ {
sock_t sock; sock_t sock;
version (linux) { version (linux) {
() @trusted { sock = socket(family, type | SOCK_NONBLOCK, 0); } (); () @trusted { sock = socket(family, type | SOCK_NONBLOCK | SOCK_CLOEXEC, 0); } ();
if (sock == -1) return -1; if (sock == -1) return -1;
} else { } else {
() @trusted { sock = socket(family, type, 0); } (); () @trusted { sock = socket(family, type, 0); } ();
if (sock == -1) return -1; if (sock == -1) return -1;
setSocketNonBlocking(cast(SocketFD)sock); setSocketNonBlocking(cast(SocketFD)sock, true);
} }
return sock; return sock;
} }
@ -917,13 +928,15 @@ private void closeSocket(sock_t sockfd)
else close(sockfd); else close(sockfd);
} }
private void setSocketNonBlocking(SocketFD sockfd) private void setSocketNonBlocking(SocketFD sockfd, bool close_on_exec = false)
@nogc nothrow { @nogc nothrow {
version (Windows) { version (Windows) {
uint enable = 1; uint enable = 1;
() @trusted { ioctlsocket(sockfd, FIONBIO, &enable); } (); () @trusted { ioctlsocket(sockfd, FIONBIO, &enable); } ();
} else { } else {
() @trusted { fcntl(cast(int)sockfd, F_SETFL, O_NONBLOCK, 1); } (); int f = O_NONBLOCK;
if (close_on_exec) f |= O_CLOEXEC;
() @trusted { fcntl(cast(int)sockfd, F_SETFL, f); } ();
} }
} }

View file

@ -6,14 +6,20 @@ module test;
import eventcore.core; import eventcore.core;
import eventcore.socket; import eventcore.socket;
import eventcore.internal.utils : print;
import std.socket : InternetAddress; import std.socket : InternetAddress;
import core.time : Duration; import core.time : Duration, msecs;
ubyte[256] s_rbuf; ubyte[256] s_rbuf;
bool s_done; bool s_done;
void main() void main()
{ {
// watchdog timer in case of starvation/deadlocks
auto tm = eventDriver.timers.create();
eventDriver.timers.set(tm, 10000.msecs, 0.msecs);
eventDriver.timers.wait(tm, (tm) { assert(false, "Test hung."); });
static ubyte[] pack1 = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9]; static ubyte[] pack1 = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9];
static ubyte[] pack2 = [4, 3, 2, 1, 0]; static ubyte[] pack2 = [4, 3, 2, 1, 0];
@ -24,33 +30,46 @@ void main()
server.waitForConnections!((incoming_, addr) { server.waitForConnections!((incoming_, addr) {
incoming = incoming_; // work around ref counting issue incoming = incoming_; // work around ref counting issue
print("Got incoming, reading data");
incoming.read!((status, bts) { incoming.read!((status, bts) {
print("Got data");
assert(status == IOStatus.ok); assert(status == IOStatus.ok);
assert(bts == pack1.length); assert(bts == pack1.length);
assert(s_rbuf[0 .. pack1.length] == pack1); assert(s_rbuf[0 .. pack1.length] == pack1);
print("Second write");
client.write!((status, bytes) { client.write!((status, bytes) {
print("Second write done");
assert(status == IOStatus.ok); assert(status == IOStatus.ok);
assert(bytes == pack2.length); assert(bytes == pack2.length);
})(pack2, IOMode.once); })(pack2, IOMode.once);
print("Second read");
incoming.read!((status, bts) { incoming.read!((status, bts) {
print("Second read done");
assert(status == IOStatus.ok); assert(status == IOStatus.ok);
assert(bts == pack2.length); assert(bts == pack2.length);
assert(s_rbuf[0 .. pack2.length] == pack2); assert(s_rbuf[0 .. pack2.length] == pack2);
destroy(client);
destroy(incoming); destroy(incoming);
destroy(server); destroy(server);
destroy(client);
s_done = true; s_done = true;
eventDriver.timers.stop(tm);
// NOTE: one reference to incoming is still held by read()
//assert(eventDriver.core.waiterCount == 1);
})(s_rbuf, IOMode.once); })(s_rbuf, IOMode.once);
})(s_rbuf, IOMode.once); })(s_rbuf, IOMode.once);
}); });
print("Connect...");
connectStream!((sock, status) { connectStream!((sock, status) {
client = sock; client = sock;
assert(status == ConnectStatus.connected); assert(status == ConnectStatus.connected);
print("Initial write");
client.write!((wstatus, bytes) { client.write!((wstatus, bytes) {
print("Initial write done");
assert(wstatus == IOStatus.ok); assert(wstatus == IOStatus.ok);
assert(bytes == 10); assert(bytes == 10);
})(pack1, IOMode.all); })(pack1, IOMode.all);