speedtch usbatm.h,1.4,1.5 usbatm2.c,1.13,1.14

matthieu castet castet.matthieu at free.fr
Sat Feb 12 09:50:26 EST 2005


Hi,
Duncan Sands wrote:
> Hi Roman,
> 
> 
>>If it doesn't go in, I think we at least should remove that bogus kfree call
>>in udsl_destroy_instance: a memleak is better than a probability of a
>>crash.
> 
> 
> which bogus kfree do you mean?  I see a kfree in udsl_destroy_instance,
> but it looks fine to me...

the kfree is done on data which is statically initialized by the driver, 
so the kfree is is bogus.
But even if we remove the kfree, it won't solve the problem, since we 
can unload the driver and erase the instance data(loading an other 
module that reuse the memory where the instance was stored), and if 
usbatm try to access to this (from atm call for example) it with panic...


Matthieu



More information about the Usbatm mailing list