[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] pciback: check function return values



>>> Chris Bookholt <cgbook2@xxxxxxxxxxxxxx> 03.04.07 17:56 >>>
>Jan Beulich wrote:
>> .. of functions declared with __must_check post-2.6.18.
>> 
>> (As a note - I'm getting the impression that when used as a module, pciback
>> appears to have a number of memory leaks).
>
>Jan, thanks for the added robustness by way of return value checking.
>
>What gives you the impression there are memory leaks when pciback is
>loaded as a module?  The more specific you can be the easier problems
>are to find and fix.

First thing pcistub_init() does is parse the hide option, calling
pcistub_device_id_add(). If subsequent steps fail, or if the driver is unloaded,
this isn't being cleaned up. There are others that look suspicious, but this is
certainly easier to determine for someone more familiar with the code than
I am.

Jan

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.