diff options
author | Rich Felker <dalias@aerifal.cx> | 2013-06-28 23:57:58 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2013-06-28 23:57:58 -0400 |
commit | 062f40ef3e56021f4a9902095867e35cce6d99c4 (patch) | |
tree | 102a798592f8fdc5c74acdec9d8cb673b822cb93 /include/sys/sem.h | |
parent | 553d566c3f7080cf1f339eebf715db7e5d0b0d76 (diff) | |
download | musl-062f40ef3e56021f4a9902095867e35cce6d99c4.tar.gz musl-062f40ef3e56021f4a9902095867e35cce6d99c4.tar.xz musl-062f40ef3e56021f4a9902095867e35cce6d99c4.zip |
work around wrong kernel type for sem_nsems member of struct semid_ds
rejecting invalid values for n is fine even in the case where a new sem will not be created, since the kernel does its range checks on n even in this case as well. by default, the kernel will bound the limit well below USHRT_MAX anyway, but it's presumably possible that an administrator could override this limit and break things.
Diffstat (limited to 'include/sys/sem.h')
-rw-r--r-- | include/sys/sem.h | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/include/sys/sem.h b/include/sys/sem.h index cc3a3e63..e74ea208 100644 --- a/include/sys/sem.h +++ b/include/sys/sem.h @@ -25,13 +25,21 @@ extern "C" { #define SETVAL 16 #define SETALL 17 +#include <endian.h> + struct semid_ds { struct ipc_perm sem_perm; long sem_otime; unsigned long __unused1; long sem_ctime; unsigned long __unused2; - unsigned long sem_nsems; +#if __BYTE_ORDER == __LITTLE_ENDIAN + unsigned short sem_nsems; + char __sem_nsems_pad[sizeof(long)-sizeof(short)]; +#else + char __sem_nsems_pad[sizeof(long)-sizeof(short)]; + unsigned short sem_nsems; +#endif unsigned long __unused3; unsigned long __unused4; }; |