[tor-commits] [tor/maint-0.3.2] Fix memory leak in client_likes_consensus().

[nickm at torproject.org]

commit d86c45bf5cc75a526b884a754d72ef4d11aa0693
Author: Alexander Færøy <ahf at torproject.org>
Date:   Sat Jun 23 10:33:40 2018 +0200

    Fix memory leak in client_likes_consensus().
    
    This patches fixes a memory leak in client_likes_consensus() where if
    consensus_cache_entry_get_voter_id_digests() would fail we would return
    without having free'd the voters list.
    
    See: Coverity CID 1437447
---
 src/or/directory.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/or/directory.c b/src/or/directory.c
index 5ceea2fb3..f52354356 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -3499,6 +3499,7 @@ client_likes_consensus(const struct consensus_cache_entry_t *ent,
   int have = 0;
 
   if (consensus_cache_entry_get_voter_id_digests(ent, voters) != 0) {
+    smartlist_free(voters);
     return 1; // We don't know the voters; assume the client won't mind. */
   }