Correctly free SSL_SESSIONs in ssl_test.

That was silly.

Change-Id: I375c04f725cbb75f9e04fce386e20c4de5e7ae0c
Reviewed-on: https://boringssl-review.googlesource.com/6352
Reviewed-by: Adam Langley <alangley@gmail.com>

ssl/ssl_test.cc

diff --git a/ssl/ssl_test.cc b/ssl/ssl_test.cc
index a5d8bea..345e5dc 100644
--- a/ssl/ssl_test.cc
+++ b/ssl/ssl_test.cc
@@ -877,19 +877,19 @@
 
 // TODO(davidben): Switch this to a |std::vector<ScopedSSL_SESSION>| once we can
 // rely on a move-aware |std::vector|.
-template <class T>
-class ScopedVector {
+class ScopedSessionVector {
  public:
-  explicit ScopedVector(std::vector<T*> *v) : v_(v) {}
+  explicit ScopedSessionVector(std::vector<SSL_SESSION*> *sessions)
+      : sessions_(sessions) {}
 
-  ~ScopedVector() {
-    for (T *t : *v_) {
-      delete t;
+  ~ScopedSessionVector() {
+    for (SSL_SESSION *session : *sessions_) {
+      SSL_SESSION_free(session);
     }
   }
 
  private:
-  std::vector<T*> *const v_;
+  std::vector<SSL_SESSION*> *const sessions_;
 };
 
 // Test that the internal session cache behaves as expected.
@@ -901,7 +901,7 @@
 
   // Prepare 10 test sessions.
   std::vector<SSL_SESSION*> sessions;
-  ScopedVector<SSL_SESSION> cleanup(&sessions);
+  ScopedSessionVector cleanup(&sessions);
   for (int i = 0; i < 10; i++) {
     ScopedSSL_SESSION session = CreateTestSession(i);
     if (!session) {