misc/adsb-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Switch from dup() + fcntl(F_SETFL, FD_CLOEXEC) to fcntl(F_DUPFD_CLOEXEC) to save a second syscall.

Browse Source
This commit is contained in:
Ian Gulliver
2016-02-29 22:03:04 -08:00
parent b106887617
commit a093b8a1b6
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
adsbus/opts.c
View File

@@ -133,8 +133,8 @@ bool opts_add_exec_send(char *arg) {
} }
bool opts_add_stdin(char __attribute__((unused)) *arg) { bool opts_add_stdin(char __attribute__((unused)) *arg) {
int fd = dup(0); int fd = fcntl(0, F_DUPFD_CLOEXEC, 0);
assert(!fcntl(fd, F_SETFD, FD_CLOEXEC)); assert(fd >= 0);
file_fd_new(fd, receive_flow, NULL); file_fd_new(fd, receive_flow, NULL);
return true; return true;
} }
@@ -144,8 +144,8 @@ bool opts_add_stdout(char *arg) {
if (!serializer) { if (!serializer) {
return false; return false;
} }
int fd = dup(1); int fd = fcntl(1, F_DUPFD_CLOEXEC, 0);
assert(!fcntl(fd, F_SETFD, FD_CLOEXEC)); assert(fd >= 0);
file_fd_new(fd, send_flow, serializer); file_fd_new(fd, send_flow, serializer);
return true; return true;
} }