'new_interfaces' should be freed to avoid memory leak.

Cc: Sarah Sharp <sarah.a.sh...@linux.intel.com>
Cc: Alan Stern <st...@rowland.harvard.edu>
Signed-off-by: Sachin Kamat <sachin.ka...@linaro.org>
---
 drivers/usb/core/message.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/core/message.c b/drivers/usb/core/message.c
index a557658..1653f56 100644
--- a/drivers/usb/core/message.c
+++ b/drivers/usb/core/message.c
@@ -1795,7 +1795,8 @@ free_interfaces:
        if (dev->actconfig && usb_disable_lpm(dev)) {
                dev_err(&dev->dev, "%s Failed to disable LPM\n.", __func__);
                mutex_unlock(hcd->bandwidth_mutex);
-               return -ENOMEM;
+               ret = -ENOMEM;
+               goto free_interfaces;
        }
        ret = usb_hcd_alloc_bandwidth(dev, cp, NULL, NULL);
        if (ret < 0) {
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to