From 6e46962db9caab269ae620d85d526f896781a9c1 Mon Sep 17 00:00:00 2001 From: miod <> Date: Sat, 14 Feb 2015 15:29:29 +0000 Subject: Unchecked allocations in x509_name_canon(). ok doug@ jsing@ --- src/lib/libcrypto/asn1/x_name.c | 6 +++++- src/lib/libssl/src/crypto/asn1/x_name.c | 6 +++++- 2 files changed, 10 insertions(+), 2 deletions(-) (limited to 'src/lib') diff --git a/src/lib/libcrypto/asn1/x_name.c b/src/lib/libcrypto/asn1/x_name.c index c7f4759088..51c5a0ae41 100644 --- a/src/lib/libcrypto/asn1/x_name.c +++ b/src/lib/libcrypto/asn1/x_name.c @@ -1,4 +1,4 @@ -/* $OpenBSD: x_name.c,v 1.28 2015/02/14 15:25:08 jsing Exp $ */ +/* $OpenBSD: x_name.c,v 1.29 2015/02/14 15:29:29 miod Exp $ */ /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) * All rights reserved. * @@ -454,7 +454,11 @@ x509_name_canon(X509_NAME *a) set = entry->set; } tmpentry = X509_NAME_ENTRY_new(); + if (tmpentry == NULL) + goto err; tmpentry->object = OBJ_dup(entry->object); + if (tmpentry->object == NULL) + goto err; if (!asn1_string_canon(tmpentry->value, entry->value)) goto err; if (!sk_X509_NAME_ENTRY_push(entries, tmpentry)) diff --git a/src/lib/libssl/src/crypto/asn1/x_name.c b/src/lib/libssl/src/crypto/asn1/x_name.c index c7f4759088..51c5a0ae41 100644 --- a/src/lib/libssl/src/crypto/asn1/x_name.c +++ b/src/lib/libssl/src/crypto/asn1/x_name.c @@ -1,4 +1,4 @@ -/* $OpenBSD: x_name.c,v 1.28 2015/02/14 15:25:08 jsing Exp $ */ +/* $OpenBSD: x_name.c,v 1.29 2015/02/14 15:29:29 miod Exp $ */ /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) * All rights reserved. * @@ -454,7 +454,11 @@ x509_name_canon(X509_NAME *a) set = entry->set; } tmpentry = X509_NAME_ENTRY_new(); + if (tmpentry == NULL) + goto err; tmpentry->object = OBJ_dup(entry->object); + if (tmpentry->object == NULL) + goto err; if (!asn1_string_canon(tmpentry->value, entry->value)) goto err; if (!sk_X509_NAME_ENTRY_push(entries, tmpentry)) -- cgit v1.2.3-55-g6feb