From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S267352AbUG1XiH (ORCPT ); Wed, 28 Jul 2004 19:38:07 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S267327AbUG1Xec (ORCPT ); Wed, 28 Jul 2004 19:34:32 -0400 Received: from fw.osdl.org ([65.172.181.6]:40390 "EHLO mail.osdl.org") by vger.kernel.org with ESMTP id S267351AbUG1Xbe (ORCPT ); Wed, 28 Jul 2004 19:31:34 -0400 Date: Wed, 28 Jul 2004 16:34:40 -0700 From: Andrew Morton To: Johannes Stezenbach Cc: viro@parcelfarce.linux.theplanet.co.uk, linux-kernel@vger.kernel.org Subject: Re: 2.6.8-rc2-mm1 Message-Id: <20040728163440.395b22e0.akpm@osdl.org> In-Reply-To: <20040728232453.GA6377@convergence.de> References: <20040728020444.4dca7e23.akpm@osdl.org> <20040728222455.GC5878@convergence.de> <20040728224423.GJ12308@parcelfarce.linux.theplanet.co.uk> <20040728232453.GA6377@convergence.de> X-Mailer: Sylpheed version 0.9.7 (GTK+ 1.2.10; i586-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Johannes Stezenbach wrote: > > This is a hack introduced by someone years ago. The "pointer" is > actually an integer argument, e.g. in include/linux/dvb/audio.h: > > #define AUDIO_SET_MUTE _IO('o', 6) > > actually takes an integer argument (!0 mute, 0 unmute), so one can write > > if (ioctl(fd, AUDIO_SET_MUTE, 1) == -1) > perror("mute"); Is it a boolean argument? If so, we could change the code to do parg = (void *)(arg ? 1 : 0); so if someone dereferences it they'll get a nice oops.