commit 18183de0609a45a92b69d00e37ab569921e63413
Author: Nick Mathewson <nickm(a)torproject.org>
Date: Tue Aug 21 09:50:48 2018 -0400
Check for duplicate-close in connection_dir_finished_flusing()
Fix for 26896.
---
changes/bug26896 | 3 +++
src/or/directory.c | 4 +++-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/changes/bug26896 b/changes/bug26896
new file mode 100644
index 000000000..9762dc7fa
--- /dev/null
+++ b/changes/bug26896
@@ -0,0 +1,3 @@
+ o Minor bugfixes (directory connection shutdown):
+ - Avoid a double-close when shutting down a stalled directory connection.
+ Fixes bug 26896; bugfix on 0.3.4.1-alpha.
diff --git a/src/or/directory.c b/src/or/directory.c
index e763de268..32ec876f7 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -5179,6 +5179,9 @@ connection_dir_finished_flushing(dir_connection_t *conn)
tor_assert(conn);
tor_assert(conn->base_.type == CONN_TYPE_DIR);
+ if (conn->marked_for_close)
+ return 0;
+
/* Note that we have finished writing the directory response. For direct
* connections this means we're done; for tunneled connections it's only
* an intermediate step. */
@@ -5912,4 +5915,3 @@ dir_split_resource_into_spoolable(const char *resource,
smartlist_free(fingerprints);
return r;
}
-