diff options
author | Rich Felker <dalias@aerifal.cx> | 2012-06-20 22:16:47 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2012-06-20 22:16:47 -0400 |
commit | 4e8b0938d90793d6e1e200d6b25e6581b72bd4d0 (patch) | |
tree | cb0255413023716592eebc9f7978d7c1a1b00179 /src | |
parent | e5fb6820a42a1f675ba09c15273953e1ace65777 (diff) | |
download | musl-4e8b0938d90793d6e1e200d6b25e6581b72bd4d0.tar.gz |
proper error handling for fcntl F_GETOWN on modern kernels
on old kernels, there's no way to detect errors; we must assume
negative syscall return values are pgrp ids. but if the F_GETOWN_EX
fcntl works, we can get a reliable answer.
Diffstat (limited to 'src')
-rw-r--r-- | src/fcntl/fcntl.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/fcntl/fcntl.c b/src/fcntl/fcntl.c index fa5ad32f..fb7806a3 100644 --- a/src/fcntl/fcntl.c +++ b/src/fcntl/fcntl.c @@ -1,6 +1,8 @@ +#define _GNU_SOURCE #include <fcntl.h> #include <unistd.h> #include <stdarg.h> +#include <errno.h> #include "syscall.h" #include "libc.h" @@ -13,6 +15,12 @@ int fcntl(int fd, int cmd, ...) va_end(ap); if (cmd == F_SETFL) arg |= O_LARGEFILE; if (cmd == F_SETLKW) return syscall_cp(SYS_fcntl, fd, cmd, arg); - if (cmd == F_GETOWN) return __syscall(SYS_fcntl, fd, cmd, arg); + if (cmd == F_GETOWN) { + struct f_owner_ex ex; + int ret = __syscall(SYS_fcntl, fd, F_GETOWN_EX, &ex); + if (ret == -EINVAL) return __syscall(SYS_fcntl, fd, cmd, arg); + if (ret) return __syscall_ret(ret); + return ex.type == F_OWNER_PGRP ? -ex.pid : ex.pid; + } return syscall(SYS_fcntl, fd, cmd, arg); } |