Daniel Stenberg <dan...@haxx.se> writes:

> On Mon, 30 Mar 2009, Simon Josefsson wrote:
>
>> I gave up on dmalloc a long time ago, in my experience valgrind leads to 
>> better results and doesn't require changes to the build.
>>
>> In most of my other projects, I always set up so that the self-tests are run 
>> under valgrind automatically (if valgrind is installed).  Maybe that would 
>> help libssh2?  We need more self-tests, though...
>
> Yes, we need more self-tests and running them with valgrind is a good way to 
> catch most of the problems. But there are two buts here that our current leak 
> (Daniel Johnson's report) shows us where just relying on valgrind isn't good 
> enough:
>
>   A) valgrind slows down the execution a lot. I can get the leak to occur in 
> my
>      tests but it seems virtually impossible to make happen when valgrind
>      monitors/slows down the code. A plain memory-leak detection would be
>      almost no extra overhead.
>
>   B) when people detect leaks on non-valgrind platforms

Sure.  More testing can never hurt.  As long as it is built
optionally.. ;)

I would worry about A) indicating some more worry-some problem though,
it smells like a race issue?

/Simon

------------------------------------------------------------------------------
_______________________________________________
libssh2-devel mailing list
libssh2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libssh2-devel

Reply via email to