usb: gadget: don't leak configs when unbinding
authorStephen Warren <swarren@nvidia.com>
Sat, 5 Sep 2015 04:03:42 +0000 (22:03 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 11 Sep 2015 21:15:21 +0000 (17:15 -0400)
commit44bfb43f9afab9b2944bb98bc0d514303f029ff9
treeae895e7f8c96d96912525305c818cfa49b11f7c3
parent0a04ed86cfaa36d93d55016d0b3dc3f06352a2fe
usb: gadget: don't leak configs when unbinding

By the time g_dnl_unbind() is run, cdev->config has been set to NULL,
so the free() there does nothing, and the config struct is leaked.
Equally, struct usb_gadget contains a linked list of config structs, so
the code should iterate over them all and free each one, rather than
freeing one particular config struct.

composite_unbind() already iterates over the list of config structs, and
unlinks each from the linked list. Fix this loop to free() each struct as
it's unlinked and otherwise forgotten.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Acked-by: Lukasz Majewski <l.majewski@samsung.com>
Tested-by: Lukasz Majewski <l.majewski@samsung.com>
drivers/usb/gadget/composite.c
drivers/usb/gadget/g_dnl.c