The branch master has been updated via 512d811719fc955f574090af4c3586a9aba46fa7 (commit) from d689f313cc5ebe2763fb26d1e7f8cd53b7da6a4a (commit)
- Log ----------------------------------------------------------------- commit 512d811719fc955f574090af4c3586a9aba46fa7 Author: Matt Caswell <m...@openssl.org> Date: Mon Sep 10 16:23:14 2018 +0100 Check the return value from ASN1_INTEGER_set Found by Coverity Reviewed-by: Richard Levitte <levi...@openssl.org> (Merged from https://github.com/openssl/openssl/pull/7169) ----------------------------------------------------------------------- Summary of changes: crypto/pkcs12/p12_init.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/crypto/pkcs12/p12_init.c b/crypto/pkcs12/p12_init.c index a78e183..7b76909 100644 --- a/crypto/pkcs12/p12_init.c +++ b/crypto/pkcs12/p12_init.c @@ -22,7 +22,8 @@ PKCS12 *PKCS12_init(int mode) PKCS12err(PKCS12_F_PKCS12_INIT, ERR_R_MALLOC_FAILURE); return NULL; } - ASN1_INTEGER_set(pkcs12->version, 3); + if (!ASN1_INTEGER_set(pkcs12->version, 3)) + goto err; pkcs12->authsafes->type = OBJ_nid2obj(mode); switch (mode) { case NID_pkcs7_data: _____ openssl-commits mailing list To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits