aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--changes/bug65305
-rw-r--r--src/or/routerparse.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/changes/bug6530 b/changes/bug6530
new file mode 100644
index 000000000..825bbb752
--- /dev/null
+++ b/changes/bug6530
@@ -0,0 +1,5 @@
+ o Major security fixes:
+ - Avoid a read of uninitializd RAM when reading a vote or consensus
+ document with an unrecognized flavor name. This could lead to a
+ remote crash bug. Fixes bug 6530; bugfix on 0.2.2.6-alpha.
+
diff --git a/src/or/routerparse.c b/src/or/routerparse.c
index 4231a17c6..970353a43 100644
--- a/src/or/routerparse.c
+++ b/src/or/routerparse.c
@@ -2853,7 +2853,7 @@ networkstatus_parse_vote_from_string(const char *s, const char **eos_out,
int flavor = networkstatus_parse_flavor_name(tok->args[1]);
if (flavor < 0) {
log_warn(LD_DIR, "Can't parse document with unknown flavor %s",
- escaped(tok->args[2]));
+ escaped(tok->args[1]));
goto err;
}
ns->flavor = flav = flavor;