[or-cvs] r18299: {tor} Forward-port: Fix bug 893: check AP connections for markedne (in tor/trunk: . src/or)

[nickm at seul.org]

Author: nickm
Date: 2009-01-28 11:30:06 -0500 (Wed, 28 Jan 2009)
New Revision: 18299

Modified:
   tor/trunk/ChangeLog
   tor/trunk/src/or/connection_edge.c
Log:
Forward-port: Fix bug 893: check AP connections for markedness before expiring them.

Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2009-01-28 16:29:58 UTC (rev 18298)
+++ tor/trunk/ChangeLog	2009-01-28 16:30:06 UTC (rev 18299)
@@ -13,6 +13,8 @@
       platforms.  Bugfix on 0.2.1.1-alpha.
     - Fix code so authorities _actually_ send back X-Descriptor-Not-New
       headers.  Bugfix on 0.2.0.10-alpha.
+    - Don't consider consider expiring already-closed client connections.
+      Fixes bug 893.  Bugfix on 0.0.2pre20.
 
   o Minor features:
     - Support platforms where time_t is 64 bits long. (Congratulations,

Modified: tor/trunk/src/or/connection_edge.c
===================================================================
--- tor/trunk/src/or/connection_edge.c	2009-01-28 16:29:58 UTC (rev 18298)
+++ tor/trunk/src/or/connection_edge.c	2009-01-28 16:30:06 UTC (rev 18299)
@@ -401,7 +401,7 @@
   smartlist_t *conns = get_connection_array();
 
   SMARTLIST_FOREACH_BEGIN(conns, connection_t *, c) {
-    if (c->type != CONN_TYPE_AP)
+    if (c->type != CONN_TYPE_AP || c->marked_for_close)
       continue;
     conn = TO_EDGE_CONN(c);
     /* if it's an internal linked connection, don't yell its status. */