[tor-commits] [tor/master] Fix a use-after-free error in cleaned-up rouerlist code.

[nickm@xxxxxxxxxxxxxx]

commit 2d4c40ee5f046ef99e530fc8808f302275c27788
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Mon Oct 20 09:04:53 2014 -0400

    Fix a use-after-free error in cleaned-up rouerlist code.
    
    Bug not in any released tor.  This is CID 1248521
---
 src/or/routerlist.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 93f9632..5748dfd 100644
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -3974,10 +3974,10 @@ router_load_extrainfo_from_string(const char *s, const char *eos,
   log_info(LD_DIR, "%d elements to add", smartlist_len(extrainfo_list));
 
   SMARTLIST_FOREACH_BEGIN(extrainfo_list, extrainfo_t *, ei) {
-      was_router_added_t added =
-        router_add_extrainfo_to_routerlist(ei, &msg, from_cache, !from_cache);
       uint8_t d[DIGEST_LEN];
       memcpy(d, ei->cache_info.signed_descriptor_digest, DIGEST_LEN);
+      was_router_added_t added =
+        router_add_extrainfo_to_routerlist(ei, &msg, from_cache, !from_cache);
       if (WRA_WAS_ADDED(added) && requested_fingerprints) {
         char fp[HEX_DIGEST_LEN+1];
         base16_encode(fp, sizeof(fp), descriptor_digests ?

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits