author: "William Lima" <william.lima@hscbrasil.com.br>

Fix memory leak when the cache of sslcrtvalidator_program is disabled via ttl=0

diff --git a/src/ssl/helper.cc b/src/ssl/helper.cc
index 741bcef4152450c80df084a49531939ea18f5e90..224bd4d0fcaea69616cf78e7dd88aa6e19c3a548 100644 (file)
--- a/src/ssl/helper.cc
+++ b/src/ssl/helper.cc
@@ -231,7 +231,8 @@ sslCrtvdHandleReplyWrapper(void *data, const ::Helper::Reply &reply)
     if (Ssl::CertValidationHelper::HelperCache &&
             (validationResponse->resultCode == ::Helper::Okay || validationResponse->resultCode == ::Helper::Error)) {
         Ssl::CertValidationResponse::Pointer *item = new Ssl::CertValidationResponse::Pointer(validationResponse);
-        Ssl::CertValidationHelper::HelperCache->add(crtdvdData->query.c_str(), item);
+        if (!Ssl::CertValidationHelper::HelperCache->add(crtdvdData->query.c_str(), item))
+            delete item;
     }
 
     SSL_free(crtdvdData->ssl);