]> git.baikalelectronics.ru Git - kernel.git/commitdiff
USB storage: audit sysfs attribute permissions
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 27 Aug 2013 20:13:07 +0000 (13:13 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 27 Aug 2013 20:13:07 +0000 (13:13 -0700)
Convert the usbsorage sysfs attribute to use the _RW macro to make it
easier to determine the permissions for the file.

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/storage/scsiglue.c

index 92b05d95ec5e41d70ef48383b53f045da0e42694..94d75edef77fdf34e4f5e341e5952e532f03f09e 100644 (file)
@@ -496,7 +496,7 @@ US_DO_ALL_FLAGS
  ***********************************************************************/
 
 /* Output routine for the sysfs max_sectors file */
-static ssize_t show_max_sectors(struct device *dev, struct device_attribute *attr, char *buf)
+static ssize_t max_sectors_show(struct device *dev, struct device_attribute *attr, char *buf)
 {
        struct scsi_device *sdev = to_scsi_device(dev);
 
@@ -504,7 +504,7 @@ static ssize_t show_max_sectors(struct device *dev, struct device_attribute *att
 }
 
 /* Input routine for the sysfs max_sectors file */
-static ssize_t store_max_sectors(struct device *dev, struct device_attribute *attr, const char *buf,
+static ssize_t max_sectors_store(struct device *dev, struct device_attribute *attr, const char *buf,
                size_t count)
 {
        struct scsi_device *sdev = to_scsi_device(dev);
@@ -514,16 +514,14 @@ static ssize_t store_max_sectors(struct device *dev, struct device_attribute *at
                blk_queue_max_hw_sectors(sdev->request_queue, ms);
                return count;
        }
-       return -EINVAL; 
+       return -EINVAL;
 }
-
-static DEVICE_ATTR(max_sectors, S_IRUGO | S_IWUSR, show_max_sectors,
-               store_max_sectors);
+static DEVICE_ATTR_RW(max_sectors);
 
 static struct device_attribute *sysfs_device_attr_list[] = {
-               &dev_attr_max_sectors,
-               NULL,
-               };
+       &dev_attr_max_sectors,
+       NULL,
+};
 
 /*
  * this defines our host template, with which we'll allocate hosts