From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ppsw-52.csi.cam.ac.uk ([131.111.8.152]:43061 "EHLO ppsw-52.csi.cam.ac.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754998Ab0KRQ6w (ORCPT ); Thu, 18 Nov 2010 11:58:52 -0500 Message-ID: <4CE55CC1.6010507@cam.ac.uk> Date: Thu, 18 Nov 2010 17:05:05 +0000 From: Jonathan Cameron MIME-Version: 1.0 To: gregkh@suse.de CC: Drivers , stable@kernel.org, torvalds@linux-foundation.org, "linux-iio@vger.kernel.org" Subject: Re: patch "Staging: iio: adis16220: fix up some sysfs attribute permissions" added to staging tree References: <12899355293477@site> In-Reply-To: <12899355293477@site> Content-Type: text/plain; charset=us-ascii Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 11/16/10 19:25, gregkh@suse.de wrote: > > This is a note to let you know that I've just added the patch titled > > Staging: iio: adis16220: fix up some sysfs attribute permissions > > to my staging git tree which can be found at > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging-2.6.git > in the staging-linus branch. > > The patch will show up in the next release of the linux-next tree > (usually sometime within the next 24 hours during the week.) > > The patch will hopefully also will be merged in Linus's tree for the > next -rc kernel release. > > If you have any questions about this process, please let me know. > > >>>>From 1d904e8950c86e670ace237eaea1d48cd81e94df Mon Sep 17 00:00:00 2001 > From: Greg Kroah-Hartman > Date: Tue, 16 Nov 2010 11:19:53 -0800 > Subject: Staging: iio: adis16220: fix up some sysfs attribute permissions > > They should not be writable by any user Sorry Greg, it may be jet lag related, but I really don't see how this change makes sense... Doesn't this take a write only parameter and make it read only with no read function defined? Was the intent to restrict who could write to this attribute? > > Reported-by: Linus Torvalds > Cc: Jonathan Cameron > Cc: Barry Song > Cc: stable > Signed-off-by: Greg Kroah-Hartman > --- > drivers/staging/iio/accel/adis16220_core.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/staging/iio/accel/adis16220_core.c b/drivers/staging/iio/accel/adis16220_core.c > index c86d149..c2b2091 100644 > --- a/drivers/staging/iio/accel/adis16220_core.c > +++ b/drivers/staging/iio/accel/adis16220_core.c > @@ -507,7 +507,7 @@ static IIO_DEVICE_ATTR(reset, S_IWUSR, NULL, > adis16220_write_reset, 0); > > #define IIO_DEV_ATTR_CAPTURE(_store) \ > - IIO_DEVICE_ATTR(capture, S_IWUGO, NULL, _store, 0) > + IIO_DEVICE_ATTR(capture, S_IRUSR, NULL, _store, 0) > > static IIO_DEV_ATTR_CAPTURE(adis16220_write_capture); >