Commit 49b5e5cf authored by Nick Mathewson's avatar Nick Mathewson 🎨
Browse files

r11610@catbus: nickm | 2007-01-30 17:44:02 -0500

 Fix a bug in our assertions related to removing recently-completed DNS lookup attempts on exit.  It is okay to find a cache entry with the same name as a DONE entry, so long it is not the same struct as the DONE entry.


svn:r9467
parent 76f896e7
......@@ -324,8 +324,8 @@ dns_free_all(void)
/* XXXX012 The hash lookups here could be quite slow; remove them
* once we're happy. */
if (res->state == CACHE_STATE_DONE) {
cached_resolve_t *removed = HT_REMOVE(cache_map, &cache_root, res);
tor_assert(!removed);
cached_resolve_t *found = HT_FIND(cache_map, &cache_root, res);
tor_assert(!found || found != res);
_free_cached_resolve(res);
} else {
cached_resolve_t *found = HT_FIND(cache_map, &cache_root, res);
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment