From 8d02e1986f2643ed0c05cf3726840f3c5f2486cd Mon Sep 17 00:00:00 2001
From: tedu <>
Date: Tue, 22 Apr 2014 20:31:57 +0000
Subject: fix memory leaks. from Dirk Engling

---
 src/lib/libcrypto/asn1/f_enum.c           | 11 ++++-------
 src/lib/libcrypto/asn1/f_int.c            | 11 ++++-------
 src/lib/libcrypto/asn1/f_string.c         | 11 ++++-------
 src/lib/libssl/src/crypto/asn1/f_enum.c   | 11 ++++-------
 src/lib/libssl/src/crypto/asn1/f_int.c    | 11 ++++-------
 src/lib/libssl/src/crypto/asn1/f_string.c | 11 ++++-------
 6 files changed, 24 insertions(+), 42 deletions(-)

diff --git a/src/lib/libcrypto/asn1/f_enum.c b/src/lib/libcrypto/asn1/f_enum.c
index 98fa312266..5c42cdff01 100644
--- a/src/lib/libcrypto/asn1/f_enum.c
+++ b/src/lib/libcrypto/asn1/f_enum.c
@@ -158,8 +158,6 @@ a2i_ASN1_ENUMERATED(BIO *bp, ASN1_ENUMERATED *bs, char *buf, int size)
 			if (sp == NULL) {
 				ASN1err(ASN1_F_A2I_ASN1_ENUMERATED,
 				    ERR_R_MALLOC_FAILURE);
-				if (s != NULL)
-					free(s);
 				goto err;
 			}
 			s = sp;
@@ -191,12 +189,11 @@ a2i_ASN1_ENUMERATED(BIO *bp, ASN1_ENUMERATED *bs, char *buf, int size)
 	}
 	bs->length = num;
 	bs->data = s;
-	ret = 1;
+	return (1);
 
-err:
-	if (0) {
 err_sl:
-		ASN1err(ASN1_F_A2I_ASN1_ENUMERATED, ASN1_R_SHORT_LINE);
-	}
+	ASN1err(ASN1_F_A2I_ASN1_ENUMERATED, ASN1_R_SHORT_LINE);
+err:
+	free(s);
 	return (ret);
 }
diff --git a/src/lib/libcrypto/asn1/f_int.c b/src/lib/libcrypto/asn1/f_int.c
index 3f671d1c49..41e0e75226 100644
--- a/src/lib/libcrypto/asn1/f_int.c
+++ b/src/lib/libcrypto/asn1/f_int.c
@@ -162,8 +162,6 @@ a2i_ASN1_INTEGER(BIO *bp, ASN1_INTEGER *bs, char *buf, int size)
 			if (sp == NULL) {
 				ASN1err(ASN1_F_A2I_ASN1_INTEGER,
 				    ERR_R_MALLOC_FAILURE);
-				if (s != NULL)
-					free(s);
 				goto err;
 			}
 			s = sp;
@@ -195,12 +193,11 @@ a2i_ASN1_INTEGER(BIO *bp, ASN1_INTEGER *bs, char *buf, int size)
 	}
 	bs->length = num;
 	bs->data = s;
-	ret = 1;
+	return (1);
 
-err:
-	if (0) {
 err_sl:
-		ASN1err(ASN1_F_A2I_ASN1_INTEGER, ASN1_R_SHORT_LINE);
-	}
+	ASN1err(ASN1_F_A2I_ASN1_INTEGER, ASN1_R_SHORT_LINE);
+err:
+	free(s);
 	return (ret);
 }
diff --git a/src/lib/libcrypto/asn1/f_string.c b/src/lib/libcrypto/asn1/f_string.c
index c213c7a88d..14f6554ab3 100644
--- a/src/lib/libcrypto/asn1/f_string.c
+++ b/src/lib/libcrypto/asn1/f_string.c
@@ -154,8 +154,6 @@ a2i_ASN1_STRING(BIO *bp, ASN1_STRING *bs, char *buf, int size)
 			if (sp == NULL) {
 				ASN1err(ASN1_F_A2I_ASN1_STRING,
 				    ERR_R_MALLOC_FAILURE);
-				if (s != NULL)
-					free(s);
 				goto err;
 			}
 			s = sp;
@@ -187,12 +185,11 @@ a2i_ASN1_STRING(BIO *bp, ASN1_STRING *bs, char *buf, int size)
 	}
 	bs->length = num;
 	bs->data = s;
-	ret = 1;
+	return (1);
 
-err:
-	if (0) {
 err_sl:
-		ASN1err(ASN1_F_A2I_ASN1_STRING, ASN1_R_SHORT_LINE);
-	}
+	ASN1err(ASN1_F_A2I_ASN1_STRING, ASN1_R_SHORT_LINE);
+err:
+	free(s);
 	return (ret);
 }
diff --git a/src/lib/libssl/src/crypto/asn1/f_enum.c b/src/lib/libssl/src/crypto/asn1/f_enum.c
index 98fa312266..5c42cdff01 100644
--- a/src/lib/libssl/src/crypto/asn1/f_enum.c
+++ b/src/lib/libssl/src/crypto/asn1/f_enum.c
@@ -158,8 +158,6 @@ a2i_ASN1_ENUMERATED(BIO *bp, ASN1_ENUMERATED *bs, char *buf, int size)
 			if (sp == NULL) {
 				ASN1err(ASN1_F_A2I_ASN1_ENUMERATED,
 				    ERR_R_MALLOC_FAILURE);
-				if (s != NULL)
-					free(s);
 				goto err;
 			}
 			s = sp;
@@ -191,12 +189,11 @@ a2i_ASN1_ENUMERATED(BIO *bp, ASN1_ENUMERATED *bs, char *buf, int size)
 	}
 	bs->length = num;
 	bs->data = s;
-	ret = 1;
+	return (1);
 
-err:
-	if (0) {
 err_sl:
-		ASN1err(ASN1_F_A2I_ASN1_ENUMERATED, ASN1_R_SHORT_LINE);
-	}
+	ASN1err(ASN1_F_A2I_ASN1_ENUMERATED, ASN1_R_SHORT_LINE);
+err:
+	free(s);
 	return (ret);
 }
diff --git a/src/lib/libssl/src/crypto/asn1/f_int.c b/src/lib/libssl/src/crypto/asn1/f_int.c
index 3f671d1c49..41e0e75226 100644
--- a/src/lib/libssl/src/crypto/asn1/f_int.c
+++ b/src/lib/libssl/src/crypto/asn1/f_int.c
@@ -162,8 +162,6 @@ a2i_ASN1_INTEGER(BIO *bp, ASN1_INTEGER *bs, char *buf, int size)
 			if (sp == NULL) {
 				ASN1err(ASN1_F_A2I_ASN1_INTEGER,
 				    ERR_R_MALLOC_FAILURE);
-				if (s != NULL)
-					free(s);
 				goto err;
 			}
 			s = sp;
@@ -195,12 +193,11 @@ a2i_ASN1_INTEGER(BIO *bp, ASN1_INTEGER *bs, char *buf, int size)
 	}
 	bs->length = num;
 	bs->data = s;
-	ret = 1;
+	return (1);
 
-err:
-	if (0) {
 err_sl:
-		ASN1err(ASN1_F_A2I_ASN1_INTEGER, ASN1_R_SHORT_LINE);
-	}
+	ASN1err(ASN1_F_A2I_ASN1_INTEGER, ASN1_R_SHORT_LINE);
+err:
+	free(s);
 	return (ret);
 }
diff --git a/src/lib/libssl/src/crypto/asn1/f_string.c b/src/lib/libssl/src/crypto/asn1/f_string.c
index c213c7a88d..14f6554ab3 100644
--- a/src/lib/libssl/src/crypto/asn1/f_string.c
+++ b/src/lib/libssl/src/crypto/asn1/f_string.c
@@ -154,8 +154,6 @@ a2i_ASN1_STRING(BIO *bp, ASN1_STRING *bs, char *buf, int size)
 			if (sp == NULL) {
 				ASN1err(ASN1_F_A2I_ASN1_STRING,
 				    ERR_R_MALLOC_FAILURE);
-				if (s != NULL)
-					free(s);
 				goto err;
 			}
 			s = sp;
@@ -187,12 +185,11 @@ a2i_ASN1_STRING(BIO *bp, ASN1_STRING *bs, char *buf, int size)
 	}
 	bs->length = num;
 	bs->data = s;
-	ret = 1;
+	return (1);
 
-err:
-	if (0) {
 err_sl:
-		ASN1err(ASN1_F_A2I_ASN1_STRING, ASN1_R_SHORT_LINE);
-	}
+	ASN1err(ASN1_F_A2I_ASN1_STRING, ASN1_R_SHORT_LINE);
+err:
+	free(s);
 	return (ret);
 }
-- 
cgit v1.2.3-55-g6feb