aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2017-06-08 09:21:15 -0400
committerNick Mathewson <nickm@torproject.org>2017-06-08 09:21:15 -0400
commit0a78d2947cc1f7a0aa2ede2c37baec3e4c23a424 (patch)
tree23975799604b44e00f5e50b0da2e863b07ec2321
parent020c003a636a876a1b86295592e8782ab93df5a3 (diff)
parentdec7998f5c42dde955760efc72f57025a32da238 (diff)
downloadtor-0a78d2947cc1f7a0aa2ede2c37baec3e4c23a424.tar.gz
tor-0a78d2947cc1f7a0aa2ede2c37baec3e4c23a424.zip
Merge branch 'maint-0.2.5' into release-0.2.5
-rw-r--r--changes/trove-2017-0057
-rw-r--r--src/or/relay.c3
2 files changed, 9 insertions, 1 deletions
diff --git a/changes/trove-2017-005 b/changes/trove-2017-005
new file mode 100644
index 0000000000..cebb013f86
--- /dev/null
+++ b/changes/trove-2017-005
@@ -0,0 +1,7 @@
+ o Major bugfixes (hidden service, relay, security):
+ - Fix an assertion failure caused by receiving a BEGIN_DIR cell on
+ a hidden service rendezvous circuit. Fixes bug 22494, tracked as
+ TROVE-2017-005 and CVE-2017-0376; bugfix on 0.2.2.1-alpha. Found
+ by armadev.
+
+
diff --git a/src/or/relay.c b/src/or/relay.c
index 9407df0559..daf354c34c 100644
--- a/src/or/relay.c
+++ b/src/or/relay.c
@@ -1500,7 +1500,8 @@ connection_edge_process_relay_cell(cell_t *cell, circuit_t *circ,
"Begin cell for known stream. Dropping.");
return 0;
}
- if (rh.command == RELAY_COMMAND_BEGIN_DIR) {
+ if (rh.command == RELAY_COMMAND_BEGIN_DIR &&
+ circ->purpose != CIRCUIT_PURPOSE_S_REND_JOINED) {
/* Assign this circuit and its app-ward OR connection a unique ID,
* so that we can measure download times. The local edge and dir
* connection will be assigned the same ID when they are created