Revert "usb: interface authorization: Use a flag for the default device authorization"
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 18 Aug 2015 16:56:04 +0000 (09:56 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 18 Aug 2015 16:56:04 +0000 (09:56 -0700)
This reverts commit 3cf1fc80655d3af7083ea4b3615e5f8532543be7 as the
signed-off-by address is invalid.

Cc: Stefan Koch <stefan.koch10@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/core/hcd.c
drivers/usb/core/usb.c
include/linux/usb/hcd.h

index 86b3d11905000bf4d248dea9bfd989eeea589574..83df1dde9c08f683449ce0d6ecc8a52ed2b93817 100644 (file)
@@ -854,10 +854,10 @@ static ssize_t authorized_default_show(struct device *dev,
 {
        struct usb_device *rh_usb_dev = to_usb_device(dev);
        struct usb_bus *usb_bus = rh_usb_dev->bus;
-       struct usb_hcd *hcd;
+       struct usb_hcd *usb_hcd;
 
-       hcd = bus_to_hcd(usb_bus);
-       return snprintf(buf, PAGE_SIZE, "%u\n", !!HCD_DEV_AUTHORIZED(hcd));
+       usb_hcd = bus_to_hcd(usb_bus);
+       return snprintf(buf, PAGE_SIZE, "%u\n", usb_hcd->authorized_default);
 }
 
 static ssize_t authorized_default_store(struct device *dev,
@@ -868,16 +868,12 @@ static ssize_t authorized_default_store(struct device *dev,
        unsigned val;
        struct usb_device *rh_usb_dev = to_usb_device(dev);
        struct usb_bus *usb_bus = rh_usb_dev->bus;
-       struct usb_hcd *hcd;
+       struct usb_hcd *usb_hcd;
 
-       hcd = bus_to_hcd(usb_bus);
+       usb_hcd = bus_to_hcd(usb_bus);
        result = sscanf(buf, "%u\n", &val);
        if (result == 1) {
-               if (val)
-                       set_bit(HCD_FLAG_DEV_AUTHORIZED, &hcd->flags);
-               else
-                       clear_bit(HCD_FLAG_DEV_AUTHORIZED, &hcd->flags);
-
+               usb_hcd->authorized_default = val ? 1 : 0;
                result = size;
        } else {
                result = -EINVAL;
@@ -2724,17 +2720,10 @@ int usb_add_hcd(struct usb_hcd *hcd,
        dev_info(hcd->self.controller, "%s\n", hcd->product_desc);
 
        /* Keep old behaviour if authorized_default is not in [0, 1]. */
-       if (authorized_default < 0 || authorized_default > 1) {
-               if (hcd->wireless)
-                       clear_bit(HCD_FLAG_DEV_AUTHORIZED, &hcd->flags);
-               else
-                       set_bit(HCD_FLAG_DEV_AUTHORIZED, &hcd->flags);
-       } else {
-               if (authorized_default)
-                       set_bit(HCD_FLAG_DEV_AUTHORIZED, &hcd->flags);
-               else
-                       clear_bit(HCD_FLAG_DEV_AUTHORIZED, &hcd->flags);
-       }
+       if (authorized_default < 0 || authorized_default > 1)
+               hcd->authorized_default = hcd->wireless ? 0 : 1;
+       else
+               hcd->authorized_default = authorized_default;
        set_bit(HCD_FLAG_HW_ACCESSIBLE, &hcd->flags);
 
        /* per default all interfaces are authorized */
index f8bbd0b6d9fe44cb0077a200588c1ea054914580..8d5b2f4113cd563004acbabb459f4e4212361138 100644 (file)
@@ -510,7 +510,7 @@ struct usb_device *usb_alloc_dev(struct usb_device *parent,
        if (root_hub)   /* Root hub always ok [and always wired] */
                dev->authorized = 1;
        else {
-               dev->authorized = !!HCD_DEV_AUTHORIZED(usb_hcd);
+               dev->authorized = usb_hcd->authorized_default;
                dev->wusb = usb_bus_is_wusb(bus) ? 1 : 0;
        }
        return dev;
index 4d147277b30d75dc698dfc8d7ab9a9e26d8b4876..36ce3d215cad1e702df02935349f17af7f8b9fab 100644 (file)
  *
  * Since "struct usb_bus" is so thin, you can't share much code in it.
  * This framework is a layer over that, and should be more sharable.
+ *
+ * @authorized_default: Specifies if new devices are authorized to
+ *                      connect by default or they require explicit
+ *                      user space authorization; this bit is settable
+ *                      through /sys/class/usb_host/X/authorized_default.
+ *                      For the rest is RO, so we don't lock to r/w it.
  */
 
 /*-------------------------------------------------------------------------*/
@@ -115,7 +121,6 @@ struct usb_hcd {
 #define HCD_FLAG_RH_RUNNING            5       /* root hub is running? */
 #define HCD_FLAG_DEAD                  6       /* controller has died? */
 #define HCD_FLAG_INTF_AUTHORIZED       7       /* authorize interfaces? */
-#define HCD_FLAG_DEV_AUTHORIZED                8       /* authorize devices? */
 
        /* The flags can be tested using these macros; they are likely to
         * be slightly faster than test_bit().
@@ -135,14 +140,6 @@ struct usb_hcd {
 #define HCD_INTF_AUTHORIZED(hcd) \
        ((hcd)->flags & (1U << HCD_FLAG_INTF_AUTHORIZED))
 
-       /*
-        * Specifies if devices are authorized by default
-        * or they require explicit user space authorization; this bit is
-        * settable through /sys/class/usb_host/X/authorized_default
-        */
-#define HCD_DEV_AUTHORIZED(hcd) \
-       ((hcd)->flags & (1U << HCD_FLAG_DEV_AUTHORIZED))
-
        /* Flags that get set only during HCD registration or removal. */
        unsigned                rh_registered:1;/* is root hub registered? */
        unsigned                rh_pollable:1;  /* may we poll the root hub? */
@@ -153,6 +150,7 @@ struct usb_hcd {
         * support the new root-hub polling mechanism. */
        unsigned                uses_new_polling:1;
        unsigned                wireless:1;     /* Wireless USB HCD */
+       unsigned                authorized_default:1;
        unsigned                has_tt:1;       /* Integrated TT in root hub */
        unsigned                amd_resume_bug:1; /* AMD remote wakeup quirk */
        unsigned                can_do_streams:1; /* HC supports streams */