From b31ef8cde5c6b40a736a88ea1354f3073f99c9b1 Mon Sep 17 00:00:00 2001
From: Seth Hall <seth@icir.org>
Date: Wed, 11 Jul 2012 10:58:57 -0400
Subject: [PATCH] Fixing memory leak.

---
 src/ssl-analyzer.pac | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/ssl-analyzer.pac b/src/ssl-analyzer.pac
index bd4b76ee98..d1ac470284 100644
--- a/src/ssl-analyzer.pac
+++ b/src/ssl-analyzer.pac
@@ -305,12 +305,12 @@ refine connection SSL_Conn += {
 							//i2t_ASN1_OBJECT(&pBuffer, length, obj)
 							// printf("extension length: %d\n", length);
 							// -1 indicates an error.
-							if ( length < 0 )
-								continue;
-
-							StringVal* value = new StringVal(length, (char*)pBuffer);
-							BifEvent::generate_x509_extension(bro_analyzer(),
-										bro_analyzer()->Conn(), ${rec.is_orig}, value);
+							if ( length >= 0 )
+								{
+								StringVal* value = new StringVal(length, (char*)pBuffer);
+								BifEvent::generate_x509_extension(bro_analyzer(),
+											bro_analyzer()->Conn(), ${rec.is_orig}, value);
+								}
 							OPENSSL_free(pBuffer);
 							}
 						}