return after error instead of plowing ahead. noticed by mancha1 at zoho

2 files changed, 2 insertions, 0 deletions

diff --git a/src/lib/libcrypto/ec/ec_ameth.c b/src/lib/libcrypto/ec/ec_ameth.c
index 6331903141..79dd11083e 100644
--- a/src/lib/libcrypto/ec/ec_ameth.c
+++ b/src/lib/libcrypto/ec/ec_ameth.c
@@ -352,6 +352,7 @@ static int eckey_priv_encode(PKCS8_PRIV_KEY_INFO *p8, const EVP_PKEY *pkey)
                 EC_KEY_set_enc_flags(ec_key, old_flags);
                 free(ep);
                 ECerr(EC_F_ECKEY_PRIV_ENCODE, ERR_R_EC_LIB);
+                return 0;
         }
         /* restore old encoding flags */
         EC_KEY_set_enc_flags(ec_key, old_flags);
diff --git a/src/lib/libssl/src/crypto/ec/ec_ameth.c b/src/lib/libssl/src/crypto/ec/ec_ameth.c
index 6331903141..79dd11083e 100644
--- a/src/lib/libssl/src/crypto/ec/ec_ameth.c
+++ b/src/lib/libssl/src/crypto/ec/ec_ameth.c
@@ -352,6 +352,7 @@ static int eckey_priv_encode(PKCS8_PRIV_KEY_INFO *p8, const EVP_PKEY *pkey)
                 EC_KEY_set_enc_flags(ec_key, old_flags);
                 free(ep);
                 ECerr(EC_F_ECKEY_PRIV_ENCODE, ERR_R_EC_LIB);
+                return 0;
         }
         /* restore old encoding flags */
         EC_KEY_set_enc_flags(ec_key, old_flags);