From 3ed4c5dc05a1954a8c68d0f7e9fb802f3155d308 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Wed, 16 May 2012 10:51:02 -0400 Subject: Correct the bulletproofing of routerlist_insert() The original code updated some variables, but forgot to remove a replaced old-routerdesc from rl->old_routers. Related to bug 1776. --- src/or/routerlist.c | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'src') diff --git a/src/or/routerlist.c b/src/or/routerlist.c index 86cdb83d9d..d21b40c57d 100644 --- a/src/or/routerlist.c +++ b/src/or/routerlist.c @@ -2815,6 +2815,13 @@ routerlist_insert(routerlist_t *rl, routerinfo_t *ri) ri->cache_info.signed_descriptor_digest, &(ri->cache_info)); if (sd_old) { + int idx = sd_old->routerlist_index; + sd_old->routerlist_index = -1; + smartlist_del(rl->old_routers, idx); + if (idx < smartlist_len(rl->old_routers)) { + signed_descriptor_t *d = smartlist_get(rl->old_routers, idx); + d->routerlist_index = idx; + } rl->desc_store.bytes_dropped += sd_old->signed_descriptor_len; sdmap_remove(rl->desc_by_eid_map, sd_old->extra_info_digest); signed_descriptor_free(sd_old); -- cgit v1.2.3-54-g00ecf