From 2ca827104db2425bdb7f854c7b20d102706d559a Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Tue, 16 May 2017 10:44:24 -0400 Subject: Report deleted consensuses as NOT_FOUND rather than AVAILABLE This bug happened because of a bogus pointer check in consdiffmgr_find_consensus(), not in any released Tor. Fixes CID 1409670. Good catch, Coverity! --- src/or/consdiffmgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/or/consdiffmgr.c b/src/or/consdiffmgr.c index 72a4f01dd8..a8df077001 100644 --- a/src/or/consdiffmgr.c +++ b/src/or/consdiffmgr.c @@ -577,7 +577,7 @@ consdiffmgr_find_consensus(struct consensus_cache_entry_t **entry_out, if (!handle) return CONSDIFF_NOT_FOUND; *entry_out = consensus_cache_entry_handle_get(handle); - if (entry_out) + if (*entry_out) return CONSDIFF_AVAILABLE; else return CONSDIFF_NOT_FOUND; -- cgit v1.2.3-54-g00ecf