diff options
| author | Kurt Cancemi <kurt@x64architecture.com> | 2015-05-07 16:12:33 -0400 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2015-05-11 12:21:43 +0100 |
| commit | 1c70c783af31d42f2dc1fc81c4759e87f9ea8f31 (patch) | |
| tree | 01fc6e806651f9819f70340ed24e5e8b749f6b1f | |
| parent | 9dff24e43a20e713f4a3fdcb35b8f1de02f0baae (diff) | |
Add missing NULL check in X509V3_parse_list()
Matt's note: I added a call to X509V3err to Kurt's original patch.
RT#3840
Signed-off-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(cherry picked from commit 344c271eb339fc2982e9a3584a94e51112d84584)
| -rw-r--r-- | crypto/x509v3/v3_utl.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/crypto/x509v3/v3_utl.c b/crypto/x509v3/v3_utl.c index 65dd1e2fb3..94aaebba3e 100644 --- a/crypto/x509v3/v3_utl.c +++ b/crypto/x509v3/v3_utl.c @@ -285,6 +285,10 @@ STACK_OF(CONF_VALUE) *X509V3_parse_list(const char *line) int state; /* We are going to modify the line so copy it first */ linebuf = BUF_strdup(line); + if (linebuf == NULL) { + X509V3err(X509V3_F_X509V3_PARSE_LIST, ERR_R_MALLOC_FAILURE); + goto err; + } state = HDR_NAME; ntmp = NULL; /* Go through all characters */ |