From bc0e5a9a48740b1472404de0f4b78ec22fc1a7b2 Mon Sep 17 00:00:00 2001 From: Vincent Richard Date: Sun, 23 Dec 2007 16:19:39 +0000 Subject: Use std::vector<> instead of new[] allocation (avoid memory leaks by using RAII idiom). --- src/net/tls/TLSSocket.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'src/net/tls/TLSSocket.cpp') diff --git a/src/net/tls/TLSSocket.cpp b/src/net/tls/TLSSocket.cpp index cc922796..fe5f2446 100644 --- a/src/net/tls/TLSSocket.cpp +++ b/src/net/tls/TLSSocket.cpp @@ -319,21 +319,19 @@ ref TLSSocket::getPeerCertificates() const gnutls_x509_crt_export(x509Certs[i], GNUTLS_X509_FMT_DER, NULL, &dataSize); - byte_t* data = new byte_t[dataSize]; + std::vector data(dataSize); gnutls_x509_crt_export(x509Certs[i], - GNUTLS_X509_FMT_DER, data, &dataSize); + GNUTLS_X509_FMT_DER, &data[0], &dataSize); ref cert = - security::cert::X509Certificate::import(data, dataSize); + security::cert::X509Certificate::import(&data[0], dataSize); if (cert != NULL) certs.push_back(cert); else error = true; - delete [] data; - gnutls_x509_crt_deinit(x509Certs[i]); } -- cgit v1.2.3