Messages in this thread | | | From | Oliver Neukum <> | Subject | Re: [PATCH] kaweth: Fix memory leak in kaweth_control() | Date | Sat, 19 Sep 2009 21:19:37 +0200 |
| |
Am Samstag, 19. September 2009 03:43:30 schrieb Kevin Cernekee: > kaweth_control() never frees the buffer that it allocates for the USB > control message. Test case: > > while :; do ifconfig eth2 down ; ifconfig eth2 up ; done > > This is a tiny buffer so it is a slow leak. If you want to speed up the > process, you can change the allocation size to e.g. 16384 bytes, and it > will consume several megabytes within a few minutes.
You are right, the patch is correct. But it has to go to netdev, as it is for a network driver. Please resend it and add Acked-by: Oliver Neukum <oliver@neukum.org>
Greg, it should go into stable, too.
Regards Oliver
| |