[gmx-developers] On a possibel bug in force.c (check_solvent)

[David van der Spoel]

Qikai Li wrote:
> Hi all,
> 
> When I browse the codes, I found that in static void check_solvent(...)
> the memory allocated in
> 
> cgid = make_invblock(cgs,cgs->nra);
> 
> is never freed, this may result in memory leak.
> 
> I hope I'm right!

You are right. In the 3.3 code the variable was not freed (but in the 
latestdevelopment code it was). Anyway, this is harmless, since the 
function is called only once.

> 
> Qikai Li
> @ICCAS
> 
> _______________________________________________
> gmx-developers mailing list
> gmx-developers at gromacs.org
> http://www.gromacs.org/mailman/listinfo/gmx-developers
> Please don't post (un)subscribe requests to the list. Use the www 
> interface or send it to gmx-developers-request at gromacs.org.


-- 
David.
________________________________________________________________________
David van der Spoel, PhD, Assoc. Prof., Molecular Biophysics group,
Dept. of Cell and Molecular Biology, Uppsala University.
Husargatan 3, Box 596,  	75124 Uppsala, Sweden
phone:	46 18 471 4205		fax: 46 18 511 755
spoel at xray.bmc.uu.se	spoel at gromacs.org   http://folding.bmc.uu.se
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++