OK. I will remove that comment and send again. Thanks. Daeseok Youn. 2014-03-07 17:14 GMT+09:00 Dan Carpenter <dan.carpenter@xxxxxxxxxx>: > On Fri, Mar 07, 2014 at 05:02:25PM +0900, Daeseok Youn wrote: >> >> oldi_buffer and write_buffer need to free when usb_alphatrack_delete() >> is called. >> >> Signed-off-by: Daeseok Youn <daeseok.youn@xxxxxxxxx> >> --- >> drivers/staging/frontier/alphatrack.c | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/staging/frontier/alphatrack.c b/drivers/staging/frontier/alphatrack.c >> index edd5cef..4d630da 100644 >> --- a/drivers/staging/frontier/alphatrack.c >> +++ b/drivers/staging/frontier/alphatrack.c >> @@ -208,6 +208,8 @@ static void usb_alphatrack_delete(struct usb_alphatrack *dev) >> kfree(dev->ring_buffer); >> kfree(dev->interrupt_in_buffer); >> kfree(dev->interrupt_out_buffer); >> + kfree(dev->oldi_buffer); >> + kfree(dev->write_buffer); >> kfree(dev); /* fixme oldi_buffer */ > ^^^^^^^^^^^^^^^^^ > Remove this comment now? > > regards, > dan carpenter > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
