diff options
| author | Adhemerval Zanella <azanella@linux.vnet.ibm.com> | 2015-01-06 07:59:04 -0500 |
|---|---|---|
| committer | Adhemerval Zanella <azanella@linux.vnet.ibm.com> | 2015-01-08 08:03:31 -0500 |
| commit | dd6e8af6ba1b8a95a7f1dc7422e5ea4ccc7fbd93 (patch) | |
| tree | 10046fc1091a66245c81fe62ef62ef52b11413ca /sysdeps/unix/sysv/linux/utimes.c | |
| parent | 28c38448de826314c1d89b18a26042fee21d7c9c (diff) | |
| download | glibc-dd6e8af6ba1b8a95a7f1dc7422e5ea4ccc7fbd93.tar.gz glibc-dd6e8af6ba1b8a95a7f1dc7422e5ea4ccc7fbd93.tar.xz glibc-dd6e8af6ba1b8a95a7f1dc7422e5ea4ccc7fbd93.zip | |
powerpc: Fix compiler warning on some syscalls
GCC 5.0 emits an warning when using sizeof on array function parameters
and powerpc internal syscall macros add a check for such cases. More
specifically, on powerpc64 and powerpc32 sysdep.h:
if (__builtin_classify_type (__arg3) != 5 && sizeof (__arg3) > 8) \
__illegally_sized_syscall_arg3 (); \
And for sysdeps/unix/sysv/linux/utimensat.c build GCC emits:
error: ‘sizeof’ on array function parameter ‘tsp’ will return size of
‘const struct timespec *’
This patch uses the address of first struct member instead of the struct
itself in syscall macro.
Diffstat (limited to 'sysdeps/unix/sysv/linux/utimes.c')
| -rw-r--r-- | sysdeps/unix/sysv/linux/utimes.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sysdeps/unix/sysv/linux/utimes.c b/sysdeps/unix/sysv/linux/utimes.c index 5423ff200f..2a1f2f90a1 100644 --- a/sysdeps/unix/sysv/linux/utimes.c +++ b/sysdeps/unix/sysv/linux/utimes.c @@ -29,7 +29,8 @@ int __utimes (const char *file, const struct timeval tvp[2]) { - return INLINE_SYSCALL (utimes, 2, file, tvp); + /* Avoid implicit array coercion in syscall macros. */ + return INLINE_SYSCALL (utimes, 2, file, &tvp[0]); } weak_alias (__utimes, utimes) |