Message ID | 1651223.ep7oZJM5CX@vostro.rjw.lan (mailing list archive) |
---|---|
State | Accepted, archived |
Headers | show |
On Mon, Apr 01, 2013 at 12:57:21AM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Remove the call to dev_pm_qos_hide_flags(), added by commit 6e30d7cb > "usb: Add driver/usb/core/(port.c,hub.h) files", from > usb_port_device_release(), because (1) it is completely unnecessary > (the flags have been removed already by the PM core during the > unregistration of the device object) and (2) it triggers a NULL > pointer dereference in sysfs_find_dirent() (dev->kobj->sd is NULL at > this point). > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Index: linux-pm/drivers/usb/core/port.c =================================================================== --- linux-pm.orig/drivers/usb/core/port.c +++ linux-pm/drivers/usb/core/port.c @@ -67,7 +67,6 @@ static void usb_port_device_release(stru { struct usb_port *port_dev = to_usb_port(dev); - dev_pm_qos_hide_flags(dev); kfree(port_dev); }