On Fri, Dec 07, 2012 at 08:42:25PM +0800, Chen Gang wrote: > Hello Greg Kroah-Hartman: > > in drivers/usb/renesas_usbhs/mod_host.c, in function usbhsh_queue_done: > > get ureq from pkt, by using the macro usbhsh_pkt_to_ureq (at line 637) > pkt is the sub-object of ureq (line 73..76, line 157..158) > > free ureq, by calling function usbhsh_ureq_free (at line 655) > use kfree to free ureq in function usbhsh_ureq_free (line 184..191) > originally ureq is call kzalloc in function usbhsh_ureq_alloc (line 171..179) > so pkt also free, too. > > still use pkt, by calling function usbhsh_endpoint_sequence_save (at line 657) > use pkt->zero in funcion usbhsh_endpoint_sequence_save (at line 243) > > > I find it through code review, please help to check this suggestion whether valid. I don't know, run the code to see if this is correct or not. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
