From f2e95a02b114d824905a5e2642287e060270515c Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Thu, 12 Mar 2015 11:10:47 +0000 Subject: [PATCH] Add malloc failure checks Add some missing checks for memory allocation failures in ca app. Reviewed-by: Tim Hudson (cherry picked from commit a561bfe944c0beba73551731cb98af70dfee3549) --- apps/ca.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/apps/ca.c b/apps/ca.c index 3215c56dbf..d64ec4f14c 100644 --- a/apps/ca.c +++ b/apps/ca.c @@ -2821,6 +2821,11 @@ int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, ASN1_GENERALIZEDTIME *comp_time = NULL; tmp = BUF_strdup(str); + if(!tmp) { + BIO_printf(bio_err, "memory allocation failure\n"); + goto err; + } + p = strchr(tmp, ','); rtime_str = tmp; @@ -2838,6 +2843,10 @@ int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, if (prevtm) { *prevtm = ASN1_UTCTIME_new(); + if(!*prevtm) { + BIO_printf(bio_err, "memory allocation failure\n"); + goto err; + } if (!ASN1_UTCTIME_set_string(*prevtm, rtime_str)) { BIO_printf(bio_err, "invalid revocation date %s\n", rtime_str); goto err; @@ -2878,6 +2887,10 @@ int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, goto err; } comp_time = ASN1_GENERALIZEDTIME_new(); + if(!comp_time) { + BIO_printf(bio_err, "memory allocation failure\n"); + goto err; + } if (!ASN1_GENERALIZEDTIME_set_string(comp_time, arg_str)) { BIO_printf(bio_err, "invalid compromised time %s\n", arg_str); goto err; -- 2.34.1