diff options
| author | Lennart Poettering <lennart@poettering.net> | 2015-02-10 21:25:28 +0100 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2015-02-10 21:25:38 +0100 |
| commit | 699eee62d1b1ad8af1ca944a1b500b6c3096de0d (patch) | |
| tree | a0cae7e64b138677f3230ec4886a7c39947e8b1b /CODING_STYLE | |
| parent | d6d810fbf8071f8510450dbacd1d083f37603656 (diff) | |
CODING_STYLE: elaborate on O_CLOEXEC a bit
Diffstat (limited to 'CODING_STYLE')
| -rw-r--r-- | CODING_STYLE | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/CODING_STYLE b/CODING_STYLE index 30d24e56a..1d1d97049 100644 --- a/CODING_STYLE +++ b/CODING_STYLE @@ -195,4 +195,10 @@ - When you allocate a file descriptor, it should be made O_CLOEXEC right from the beginning, as none of our files should leak to forked binaries by default. Hence, whenever you open a file, O_CLOEXEC must - be specified, right from the beginning. + be specified, right from the beginning. This also applies to + sockets. Effectively this means that all invocations to: + + a) open() must get O_CLOEXEC passed + b) socket() and socketpair() must get SOCK_CLOEXEC passed + c) recvmsg() must get MSG_CMSG_CLOEXEC set + d) F_DUPFD_CLOEXEC should be used instead of F_DUPFD, and so on |