[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[or-cvs] r16820: {tor} Coverity says it is okay to acknowledge them. Do so. Their t (tor/trunk)
Author: nickm
Date: 2008-09-09 20:06:07 -0400 (Tue, 09 Sep 2008)
New Revision: 16820
Modified:
tor/trunk/ChangeLog
Log:
Coverity says it is okay to acknowledge them. Do so. Their tool is awesome.
Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog 2008-09-09 22:19:41 UTC (rev 16819)
+++ tor/trunk/ChangeLog 2008-09-10 00:06:07 UTC (rev 16820)
@@ -17,8 +17,7 @@
- Fix compile on OpenBSD 4.4-current. Bugfix on 0.2.1.5-alpha.
Reported by Tas.
- Fixed some memory leaks --some quite frequent, some almost impossible
- to trigger-- based on results of a static analysis tool. (XXX Can we
- say which? -NM)
+ to trigger-- based on results from Coverity.
- When testing for libevent functions, set the LDFLAGS variable
correctly. (Found by Riastradh.)
- Fix an assertion bug in parsing policy-related options; possible fix
@@ -32,7 +31,8 @@
o Code simplifications and refactoring:
- Revise the connection_new functions so that a more typesafe variant
- exists. This will lower false positives from some scanning tools.
+ exists. This will work better with Coverity, and let us find any
+ actual mistakes we're making here.
- Refactor unit testing logic so that dmalloc can be used sensibly with
unit tests to check for memory leaks.