On Fri, 2012-02-10 at 21:58 +0100, Sebastian Andrzej Siewior wrote: > Those should be free() among with the others on cleanup > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> > --- > drivers/target/usb-gadget/bot.c | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/drivers/target/usb-gadget/bot.c b/drivers/target/usb-gadget/bot.c > index 999796b..cfe968d 100644 > --- a/drivers/target/usb-gadget/bot.c > +++ b/drivers/target/usb-gadget/bot.c > @@ -355,10 +355,15 @@ void bot_cleanup_old_alt(struct f_uas *fu) > usb_ep_free_request(fu->ep_in, fu->bot_req_in); > usb_ep_free_request(fu->ep_out, fu->bot_req_out); > usb_ep_free_request(fu->ep_out, fu->cmd.req); > + usb_ep_free_request(fu->ep_out, fu->bot_status.req); > + > + kfree(fu->cmd.buf); > > fu->bot_req_in = NULL; > fu->bot_req_out = NULL; > fu->cmd.req = NULL; > + fu->bot_status.req = NULL; > + fu->cmd.buf = NULL; > } > > void bot_set_alt(struct f_uas *fu) Looks fine. Applied to lio-core/master Thanks Sebastian! --nab -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
