<br><font size=2 face="sans-serif">benoit,</font>
<br>
<br><font size=2 face="sans-serif">Thanks - will fix</font>
<br>
<br><font size=2 face="sans-serif">John Forrest</font>
<br>
<br>
<br>
<table width=100%>
<tr valign=top>
<td width=40%><font size=1 face="sans-serif"><b>Benoit SIBAUD <benoit.sibaud@rd.francetelecom.com></b>
</font>
<br><font size=1 face="sans-serif">Sent by: coin-lpsolver-bounces@list.coin-or.org</font>
<p><font size=1 face="sans-serif">01/19/2006 04:41 AM</font>
<td width=59%>
<table width=100%>
<tr valign=top>
<td>
<div align=right><font size=1 face="sans-serif">To</font></div>
<td><font size=1 face="sans-serif">coin-lpsolver@list.coin-or.org</font>
<tr valign=top>
<td>
<div align=right><font size=1 face="sans-serif">cc</font></div>
<td>
<tr valign=top>
<td>
<div align=right><font size=1 face="sans-serif">Subject</font></div>
<td><font size=1 face="sans-serif">[Coin-lpsolver] Mismatch new[]/malloc
vs free in Coin/CoinMpsIO.cpp</font></table>
<br>
<table>
<tr valign=top>
<td>
<td></table>
<br></table>
<br>
<br>
<br><tt><font size=2>Hi,<br>
<br>
In Coin/CoinMpsIO.cpp, reported by valgrind on clp test exe, "Mismatched<br>
free() / delete / delete []":<br>
<br>
CoinMpsIO::copyInIntegerInformation(const char * integerType)<br>
{<br>
if (integerType) {<br>
if (!integerType_)<br>
integerType_ = new char [numberColumns_];<br>
memcpy(integerType_,integerType,numberColumns_);<br>
} else {<br>
delete [] integerType_;<br>
integerType_=NULL;<br>
}<br>
}<br>
(...)<br>
void CoinMpsIO::freeAll()<br>
{<br>
(...)<br>
free(integerType_);<br>
(...)<br>
}<br>
int CoinMpsIO::readMps(int & numberSets,CoinSet ** &sets)<br>
{<br>
(...)<br>
if (numberColumns_)<br>
integerType_ = (char *) malloc (numberColumns_*sizeof(char));<br>
else<br>
integerType_ = (char *) malloc (sizeof(char));<br>
(...)<br>
}<br>
<br>
Regards,<br>
<br>
-- <br>
Benoît Sibaud<br>
_______________________________________________<br>
Coin-lpsolver mailing list<br>
Coin-lpsolver@list.coin-or.org<br>
http://list.coin-or.org/mailman/listinfo/coin-lpsolver<br>
</font></tt>
<br>