aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLinus Nordberg <linus@torproject.org>2012-09-05 14:00:55 +0200
committerLinus Nordberg <linus@torproject.org>2012-09-10 00:43:48 +0200
commit93ee62297ff24fcd01f97c87d76953f4d5fb377c (patch)
tree32c4165191d427e01de4bc4e224c9631fdd7f352
parentf6639d5676f34234afaec9b49c89735724f7f67e (diff)
downloadtor-93ee62297ff24fcd01f97c87d76953f4d5fb377c.tar
tor-93ee62297ff24fcd01f97c87d76953f4d5fb377c.tar.gz
Don't follow the NULL pointer.
If dirvote_create_microdescriptor() returns NULL, don't use md. Found by "f. tp.". Fixes bug 6797.
-rw-r--r--changes/bug67973
-rw-r--r--src/or/dirserv.c4
2 files changed, 5 insertions, 2 deletions
diff --git a/changes/bug6797 b/changes/bug6797
new file mode 100644
index 000000000..f52f899ef
--- /dev/null
+++ b/changes/bug6797
@@ -0,0 +1,3 @@
+ o Minor bugfixes:
+ - Don't follow the NULL pointer. Found by "f. tp.". Fixes bug
+ 6797; bugfix on 0.2.4.1-alpha.
diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index 7f4690678..e7a262357 100644
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -2800,9 +2800,9 @@ dirserv_generate_networkstatus_vote_obj(crypto_pk_t *private_key,
h->microdesc_hash_line = tor_strdup(buf);
h->next = vrs->microdesc;
vrs->microdesc = h;
+ md->last_listed = now;
+ smartlist_add(microdescriptors, md);
}
- md->last_listed = now;
- smartlist_add(microdescriptors, md);
}
smartlist_add(routerstatuses, vrs);