diff options
| author | Jeremy Huddleston Sequoia <jeremyhu@apple.com> | 2016-09-11 19:47:47 -0700 |
|---|---|---|
| committer | Jeremy Huddleston Sequoia <jeremyhu@apple.com> | 2016-09-13 10:00:51 -0700 |
| commit | a5769de0f5399053e9864b753fa9755220d65ae0 (patch) | |
| tree | 0887eaf07a2813e44a4bf34efaa421c250be19cb /Xext | |
| parent | 7def2fea30060d47780dc1eedc91fada5ae1934f (diff) | |
Xext/shm: Fix usage of F_GETFD to match standard
flags = fcntl(fd, F_GETFD) is compliant.
fcntl(fd, F_GETFD, &flags) is non-compliant (Linux extension?)
cf: http://pubs.opengroup.org/onlinepubs/9699919799/functions/fcntl.html
Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
Reviewed-by: Julien Cristau <jcristau@debian.org>
Diffstat (limited to 'Xext')
| -rw-r--r-- | Xext/shm.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Xext/shm.c b/Xext/shm.c index 05575385b..125000f5f 100644 --- a/Xext/shm.c +++ b/Xext/shm.c @@ -1213,7 +1213,8 @@ shm_tmpfile(void) if (fd < 0) return -1; unlink(template); - if (fcntl(fd, F_GETFD, &flags) >= 0) { + flags = fcntl(fd, F_GETFD); + if (flags != -1) { flags |= FD_CLOEXEC; (void) fcntl(fd, F_SETFD, &flags); } |