summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2018-11-02 09:05:14 -0400
committerNick Mathewson <nickm@torproject.org>2018-11-02 09:05:14 -0400
commitf288f2478a5d455cd151e8fd13fe3ed66dfd75e5 (patch)
tree4b37ea7711f2dc91e1d517d8c74aa1155c8ccfdc
parent8ed4f1ee8418eaafcb6314f093d1186d711a837e (diff)
parent21a9d034455a9c302055524a9d4dccbfddb220f2 (diff)
downloadtor-f288f2478a5d455cd151e8fd13fe3ed66dfd75e5.tar.gz
tor-f288f2478a5d455cd151e8fd13fe3ed66dfd75e5.zip
Merge branch 'bug26896_034' into maint-0.3.4
-rw-r--r--changes/bug268963
-rw-r--r--src/or/directory.c4
2 files changed, 6 insertions, 1 deletions
diff --git a/changes/bug26896 b/changes/bug26896
new file mode 100644
index 0000000000..9762dc7fac
--- /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 e763de268f..ca53756cbe 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->base_.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;
}
-