Add error handling in dsa_main and ASN1_i2d_bio.
authorPavel Kopyl <p.kopyl@samsung.com>
Fri, 27 Oct 2017 13:13:11 +0000 (16:13 +0300)
committerBernd Edlinger <bernd.edlinger@hotmail.de>
Fri, 3 Nov 2017 14:46:51 +0000 (15:46 +0100)
CLA: trivial

Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
Reviewed-by: Bernd Edlinger <bernd.edlinger@hotmail.de>
(Merged from https://github.com/openssl/openssl/pull/4600)

apps/dsa.c
crypto/asn1/a_i2d_fp.c

index 5410f04e37eae71892f9e219bdb4328a700d7c3d..74a68ff64b47dda79d7caa04c51740518a4b4b2f 100644 (file)
@@ -217,6 +217,9 @@ int dsa_main(int argc, char **argv)
     } else if (outformat == FORMAT_MSBLOB || outformat == FORMAT_PVK) {
         EVP_PKEY *pk;
         pk = EVP_PKEY_new();
+        if (pk == NULL)
+           goto end;
+
         EVP_PKEY_set1_DSA(pk, dsa);
         if (outformat == FORMAT_PVK) {
             if (pubin) {
index c3ed991b79435c8c0a6f14366aa3abea2b047167..980c65a25d2daa9638328a78c2653efda20e0fc9 100644 (file)
@@ -38,6 +38,9 @@ int ASN1_i2d_bio(i2d_of_void *i2d, BIO *out, unsigned char *x)
     int i, j = 0, n, ret = 1;
 
     n = i2d(x, NULL);
+    if (n <= 0)
+        return 0;
+
     b = OPENSSL_malloc(n);
     if (b == NULL) {
         ASN1err(ASN1_F_ASN1_I2D_BIO, ERR_R_MALLOC_FAILURE);