aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2017-04-24 09:31:35 -0400
committerNick Mathewson <nickm@torproject.org>2017-04-24 09:31:35 -0400
commitddccc0f9b4d0cfe3de516765f575e2a73cb0ec74 (patch)
tree1b4fc50d2b9026b171f5b56840c636a448c77621
parent03e521670059c1b29f8b0068b8fe86700183b884 (diff)
parentae374e0a56014b72e54adf2afb5676ae6788124c (diff)
downloadtor-ddccc0f9b4d0cfe3de516765f575e2a73cb0ec74.tar.gz
tor-ddccc0f9b4d0cfe3de516765f575e2a73cb0ec74.zip
Merge branch 'maint-0.3.0' into release-0.3.0
-rw-r--r--changes/bug220344
-rw-r--r--src/or/control.c2
2 files changed, 5 insertions, 1 deletions
diff --git a/changes/bug22034 b/changes/bug22034
new file mode 100644
index 0000000000..6d9e188740
--- /dev/null
+++ b/changes/bug22034
@@ -0,0 +1,4 @@
+ o Minor bugfixes (control port, regression):
+ - The GETINFO extra-info/digest/<digest> command was broken because of a
+ wrong base16 decode return value check. In was introduced in a refactor
+ of that API. Fixex bug #22034; bugfix on tor-0.2.9.1-alpha.
diff --git a/src/or/control.c b/src/or/control.c
index b0a687679d..04cd8e8dc1 100644
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -2058,7 +2058,7 @@ getinfo_helper_dir(control_connection_t *control_conn,
char d[DIGEST_LEN];
signed_descriptor_t *sd = NULL;
if (base16_decode(d, sizeof(d), question, strlen(question))
- != sizeof(d)) {
+ == sizeof(d)) {
/* XXXX this test should move into extrainfo_get_by_descriptor_digest,
* but I don't want to risk affecting other parts of the code,
* especially since the rules for using our own extrainfo (including