fix type of extended argument array to pselect6 syscall
authorRich Felker <dalias@aerifal.cx>
Tue, 27 May 2014 01:26:46 +0000 (21:26 -0400)
committerRich Felker <dalias@aerifal.cx>
Tue, 27 May 2014 01:26:46 +0000 (21:26 -0400)
this only matters on x32 (and perhaps future 32-on-64 abis for other
archs); otherwise the type is long anyway. the cast through uintptr_t
prevents nonsensical "sign extension" of pointers, and follows the
principle that uintptr_t is the canonical integer type to which
pointer conversion is safe.

src/select/pselect.c

index a19e153e4583c54bedabfce0f23cd39da9699550..4e2d7b073dbad9eee66d09cdab1af9922ce6bc8b 100644 (file)
@@ -1,11 +1,12 @@
 #include <sys/select.h>
 #include <signal.h>
+#include <stdint.h>
 #include "syscall.h"
 #include "libc.h"
 
 int pselect(int n, fd_set *restrict rfds, fd_set *restrict wfds, fd_set *restrict efds, const struct timespec *restrict ts, const sigset_t *restrict mask)
 {
-       long data[2] = { (long)mask, _NSIG/8 };
+       syscall_arg_t data[2] = { (uintptr_t)mask, _NSIG/8 };
        struct timespec ts_tmp;
        if (ts) ts_tmp = *ts;
        return syscall_cp(SYS_pselect6, n, rfds, wfds, efds, ts ? &ts_tmp : 0, data);