From 5514cce54eeb6b5b2fc4bc8ec005b474e563c9e2 Mon Sep 17 00:00:00 2001 From: Marc Espie Date: Sat, 20 Nov 1999 18:52:01 +0000 Subject: ioctl arguments are u_long. Replaces `undefined' code with `defined' behavior... gcc 2.96 performs switch cases range analysis, warns about unreachable cases, and weeds them out. In switch(i) { case VALUE: } VALUE is first coerced into the type of i. If i is signed, VALUE is unsigned and exceeds i range, you lose... the result of the conversion is undefined. --- sys/compat/ibcs2/ibcs2_syscallargs.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'sys/compat/ibcs2/ibcs2_syscallargs.h') diff --git a/sys/compat/ibcs2/ibcs2_syscallargs.h b/sys/compat/ibcs2/ibcs2_syscallargs.h index 1446b718d1c..2968117aacc 100644 --- a/sys/compat/ibcs2/ibcs2_syscallargs.h +++ b/sys/compat/ibcs2/ibcs2_syscallargs.h @@ -1,4 +1,4 @@ -/* $OpenBSD: ibcs2_syscallargs.h,v 1.6 1999/06/07 07:18:35 deraadt Exp $ */ +/* $OpenBSD: ibcs2_syscallargs.h,v 1.7 1999/11/20 18:52:00 espie Exp $ */ /* * System call argument lists. @@ -189,7 +189,7 @@ struct ibcs2_sys_semsys_args { struct ibcs2_sys_ioctl_args { syscallarg(int) fd; - syscallarg(int) cmd; + syscallarg(u_long) cmd; syscallarg(caddr_t) data; }; -- cgit v1.2.3