[tor-commits] [tor/maint-0.2.1] Fix a rare memory leak in rend_cache_store

nickm at torproject.org nickm at torproject.org
Fri Jul 1 16:54:47 UTC 2011


commit 46297bc7bd86826fa79195f36059ce408ef45b6c
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Jul 1 11:52:39 2011 -0400

    Fix a rare memory leak in rend_cache_store
    
    When we rejected a descriptor for not being the one we wanted, we
    were letting the parsed descriptor go out of scope.
    
    Found by Coverity; CID # 30.
    
    Bugfix on 0.2.1.26.
    
    (No changes file yet, since this is not in any 0.2.1.x release.)
---
 src/or/rendcommon.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/or/rendcommon.c b/src/or/rendcommon.c
index 8727a70..e0c101e 100644
--- a/src/or/rendcommon.c
+++ b/src/or/rendcommon.c
@@ -1077,6 +1077,7 @@ rend_cache_store(const char *desc, size_t desc_len, int published,
     log_warn(LD_REND, "Received service descriptor for service ID %s; "
              "expected descriptor for service ID %s.",
              query, safe_str(service_id));
+    rend_service_descriptor_free(parsed);
     return -2;
   }
   now = time(NULL);





More information about the tor-commits mailing list