commit 42d65ce8d75005758e69c5f6a84684d2b3132e53
author solenberg@webrtc.org <solenberg@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> Fri Aug 01 10:07:46 2014 +0000
committer solenberg@webrtc.org <solenberg@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> Fri Aug 01 10:07:46 2014 +0000
tree c35ef4af1e62eaceb3d1578b85ae1a449775b24c
parent 1a678c61f1bc001e50771a46c5714e56238a1c10

Fix memory leak in FakeSSLCertificate::GetChain(), discovered by Linux Memcheck build/try bots.

TBR=hellner
BUG=

Review URL: https://webrtc-codereview.appspot.com/18969004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@6809 4adac7df-926f-26a2-2b94-8c16560cd09d

talk/base/fakesslidentity.h

diff --git a/talk/base/fakesslidentity.h b/talk/base/fakesslidentity.h
index fbe5e64..5864dcc 100644
--- a/talk/base/fakesslidentity.h
+++ b/talk/base/fakesslidentity.h
@@ -82,6 +82,7 @@
     std::vector<SSLCertificate*> new_certs(certs_.size());
     std::transform(certs_.begin(), certs_.end(), new_certs.begin(), DupCert);
     *chain = new SSLCertChain(new_certs);
+    std::for_each(new_certs.begin(), new_certs.end(), DeleteCert);
     return true;
   }
 
@@ -89,6 +90,7 @@
   static FakeSSLCertificate* DupCert(FakeSSLCertificate cert) {
     return cert.GetReference();
   }
+  static void DeleteCert(SSLCertificate* cert) { delete cert; }
   std::string data_;
   std::vector<FakeSSLCertificate> certs_;
   std::string digest_algorithm_;