Raphael Hertzog wrote: > Le vendredi 22 avril 2005 ? 02:07 -0700, Kevin Brown a ?crit : > > Okay, I've tracked this one down. The offending line is 1547, which > > reads: > > > > currph->quoted = currph->bind_type->quote(currph->value, > > currph->valuelen, &currph->quotedlen); > > > > This is never freed prior to re-assignment. The fix is to add: > > > > if (currph->quoted) Safefree(currph->quoted); > > > > right before it. > > Thank you for the patch, I sent it upstream. I hope it will be > integrated soon otherwise I'll just include it as a Debian-specific > patch in the mean time.
Excellent. By the way, valgrind kicks ass...it made relatively short work of this (memory leaks used to be much more annoying to track down)... -- Kevin Brown [EMAIL PROTECTED] -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
