[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.