summaryrefslogtreecommitdiffstats
path: root/apps/pkcs12.c
diff options
context:
space:
mode:
authorRichard Levitte <levitte@openssl.org>2016-09-28 23:39:18 +0200
committerRichard Levitte <levitte@openssl.org>2016-10-19 17:44:08 +0200
commitdd1abd4462e4e4fa84b8f8de2ec70375f9b0e191 (patch)
tree66ed9bf5494cf999e57f754b5fa43ccd51ffc36e /apps/pkcs12.c
parente972273194303e15f8dd7ce69dbcfa27cc024e9f (diff)
If an engine comes up explicitely, it must also come down explicitely
In apps/apps.c, one can set up an engine with setup_engine(). However, we freed the structural reference immediately, which means that for engines that don't already have a structural reference somewhere else (because it's a built in engine), we end up returning an invalid reference. Instead, the function release_engine() is added, and called at the end of the routines that call setup_engine(). Reviewed-by: Rich Salz <rsalz@openssl.org> (Merged from https://github.com/openssl/openssl/pull/1643)
Diffstat (limited to 'apps/pkcs12.c')
-rw-r--r--apps/pkcs12.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/apps/pkcs12.c b/apps/pkcs12.c
index 24150089fa..d8e56fbf90 100644
--- a/apps/pkcs12.c
+++ b/apps/pkcs12.c
@@ -574,6 +574,7 @@ int pkcs12_main(int argc, char **argv)
PKCS12_free(p12);
if (export_cert || inrand)
app_RAND_write_file(NULL);
+ release_engine(e);
BIO_free(in);
BIO_free_all(out);
sk_OPENSSL_STRING_free(canames);