diff options
author | Nick Mathewson <nickm@torproject.org> | 2007-06-11 22:19:40 +0000 |
---|---|---|
committer | Nick Mathewson <nickm@torproject.org> | 2007-06-11 22:19:40 +0000 |
commit | 367a77459a0f4c32e2ee75c802e7ef94d94f4efb (patch) | |
tree | 29c664aedbcefd17868407a025039b3c4281854b /src/or/routerparse.c | |
parent | f45732513ea352a5588d503d0664a33002344173 (diff) | |
download | tor-367a77459a0f4c32e2ee75c802e7ef94d94f4efb.tar tor-367a77459a0f4c32e2ee75c802e7ef94d94f4efb.tar.gz |
r13355@catbus: nickm | 2007-06-11 18:19:34 -0400
Make all the digestmaps inside routerlist_t use the new typechecking wrappers, so we can catch any more bugs like the one fixed by 10561. This turned up a bug in routerlist_remove: we were extracting a routerinfo_t from desc_digest_map. Fix that too, though it was probably harmless, since we were only doing a pointer comparison with it.
svn:r10563
Diffstat (limited to 'src/or/routerparse.c')
-rw-r--r-- | src/or/routerparse.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/or/routerparse.c b/src/or/routerparse.c index bb9c3c62b..d5670a88a 100644 --- a/src/or/routerparse.c +++ b/src/or/routerparse.c @@ -900,9 +900,10 @@ router_parse_list_from_string(const char **s, const char *eos, if (have_extrainfo && want_extrainfo) { routerlist_t *rl = router_get_routerlist(); + /* XXXX020 fix this cast to digestmap_t* */ extrainfo = extrainfo_parse_entry_from_string(*s, end, saved_location != SAVED_IN_CACHE, - rl->identity_map); + (digestmap_t*)rl->identity_map); if (extrainfo) { signed_desc = &extrainfo->cache_info; elt = extrainfo; |