On Wed, May 18, 2016 at 06:09:40PM -0400, je...@suse.com wrote:
[...]
> diff --git a/ioctl.c b/ioctl.c
> index d82af4b..f6ca2f0 100644
> --- a/ioctl.c
> +++ b/ioctl.c
> @@ -228,11 +228,20 @@ ioctl_decode(struct tcb *tcp)
>  {
>       const unsigned int code = tcp->u_arg[1];
>       const long arg = tcp->u_arg[2];
> +     int ret;
>  
>       switch (_IOC_TYPE(code)) {
>  #if defined(ALPHA) || defined(POWERPC)
> -     case 'f': case 't': case 'T':
> +     case 'f':
> +             ret = file_ioctl(tcp, code, arg);
> +             if (ret != RVAL_DECODED)
> +                     return ret;
> +     case 't'
> +     case 'T':
> +             return term_ioctl(tcp, code, arg);
>  #else /* !ALPHA */
> +     case 'f':
> +             return file_ioctl(tcp, code, arg);
>       case 0x54:
>  #endif /* !ALPHA */
>               return term_ioctl(tcp, code, arg);

This obviously makes gcc -Werror unhappy on !(ALPHA || POWERPC).
Try to configure build with --enable-gcc-Werror.


-- 
ldv

Attachment: pgp4fi1DiaOQ7.pgp
Description: PGP signature

------------------------------------------------------------------------------
Mobile security can be enabling, not merely restricting. Employees who
bring their own devices (BYOD) to work are irked by the imposition of MDM
restrictions. Mobile Device Manager Plus allows you to control only the
apps on BYO-devices by containerizing them, leaving personal data untouched!
https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to