USB: core: config.c: usb_get_configuration() simplified
authorMichal Nazarewicz <mina86@mina86.com>
Sat, 17 Apr 2010 15:12:58 +0000 (17:12 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 20 May 2010 20:21:39 +0000 (13:21 -0700)
usb_gat_configuratio() used two pointers to point to the same
memory.  Code simplified, by removing one of them.

Signed-off-by: Michal Nazarewicz <mina86@mina86.com>
Cc: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/core/config.c

index 16c1157be3fcf225e879ae35bb52aebb5dd9a4a1..e4909c26becb1922df8f66f1bcc522cd06d2dbcd 100644 (file)
@@ -735,7 +735,6 @@ int usb_get_configuration(struct usb_device *dev)
        int ncfg = dev->descriptor.bNumConfigurations;
        int result = 0;
        unsigned int cfgno, length;
-       unsigned char *buffer;
        unsigned char *bigbuffer;
        struct usb_config_descriptor *desc;
 
@@ -764,17 +763,16 @@ int usb_get_configuration(struct usb_device *dev)
        if (!dev->rawdescriptors)
                goto err2;
 
-       buffer = kmalloc(USB_DT_CONFIG_SIZE, GFP_KERNEL);
-       if (!buffer)
+       desc = kmalloc(USB_DT_CONFIG_SIZE, GFP_KERNEL);
+       if (!desc)
                goto err2;
-       desc = (struct usb_config_descriptor *)buffer;
 
        result = 0;
        for (; cfgno < ncfg; cfgno++) {
                /* We grab just the first descriptor so we know how long
                 * the whole configuration is */
                result = usb_get_descriptor(dev, USB_DT_CONFIG, cfgno,
-                   buffer, USB_DT_CONFIG_SIZE);
+                   desc, USB_DT_CONFIG_SIZE);
                if (result < 0) {
                        dev_err(ddev, "unable to read config index %d "
                            "descriptor/%s: %d\n", cfgno, "start", result);
@@ -823,7 +821,7 @@ int usb_get_configuration(struct usb_device *dev)
        result = 0;
 
 err:
-       kfree(buffer);
+       kfree(desc);
 out_not_authorized:
        dev->descriptor.bNumConfigurations = cfgno;
 err2: