From: Bodo Möller <bodo@openssl.org>
Date: Wed, 10 Jan 2001 14:26:32 +0000 (+0000)
Subject: Fix C code generate by 'openssl dsaparam -C'.
X-Git-Tag: OpenSSL_0_9_6a-beta1~81^2~54
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=c06648f7f00002c84e92cec6b819f7053401ad9c;p=oweals%2Fopenssl.git

Fix C code generate by 'openssl dsaparam -C'.
---

diff --git a/CHANGES b/CHANGES
index 77ad4d16f7..abddaa43fd 100644
--- a/CHANGES
+++ b/CHANGES
@@ -3,6 +3,10 @@
 
  Changes between 0.9.6 and 0.9.7  [xx XXX 2000]
 
+  *) Fix C code generated by 'openssl dsaparam -C': If a BN_bin2bn
+     call failed, free the DSA structure.
+     [Bodo Moeller]
+
   *) Add another call level for memory allocation routines, thereby
      allowing memory allocation callbacks that can be given file
      name and line number information.  The basic functionality
diff --git a/apps/dsaparam.c b/apps/dsaparam.c
index 5177916202..d3501c2606 100644
--- a/apps/dsaparam.c
+++ b/apps/dsaparam.c
@@ -338,7 +338,7 @@ bad:
 		printf("\tdsa->g=BN_bin2bn(dsa%d_g,sizeof(dsa%d_g),NULL);\n",
 			bits_p,bits_p);
 		printf("\tif ((dsa->p == NULL) || (dsa->q == NULL) || (dsa->g == NULL))\n");
-		printf("\t\treturn(NULL);\n");
+		printf("\t\t{ DSA_free(dsa); return(NULL); }\n");
 		printf("\treturn(dsa);\n\t}\n");
 		}