From c2cca954d749dce6972a5557d913706593a02240 Mon Sep 17 00:00:00 2001 From: miod <> Date: Tue, 7 Oct 2014 04:59:25 +0000 Subject: Use strdup() instead of malloc() + memcpy(). ok doug@ jsing@ --- src/lib/libcrypto/objects/obj_lib.c | 11 +++-------- src/lib/libssl/src/crypto/objects/obj_lib.c | 11 +++-------- 2 files changed, 6 insertions(+), 16 deletions(-) (limited to 'src/lib') diff --git a/src/lib/libcrypto/objects/obj_lib.c b/src/lib/libcrypto/objects/obj_lib.c index 6dc515e628..247bafbe01 100644 --- a/src/lib/libcrypto/objects/obj_lib.c +++ b/src/lib/libcrypto/objects/obj_lib.c @@ -1,4 +1,4 @@ -/* $OpenBSD: obj_lib.c,v 1.12 2014/07/11 08:44:49 jsing Exp $ */ +/* $OpenBSD: obj_lib.c,v 1.13 2014/10/07 04:59:25 miod Exp $ */ /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) * All rights reserved. * @@ -68,7 +68,6 @@ ASN1_OBJECT * OBJ_dup(const ASN1_OBJECT *o) { ASN1_OBJECT *r; - int i; char *ln = NULL, *sn = NULL; unsigned char *data = NULL; @@ -94,20 +93,16 @@ OBJ_dup(const ASN1_OBJECT *o) r->nid = o->nid; r->ln = r->sn = NULL; if (o->ln != NULL) { - i = strlen(o->ln) + 1; - ln = malloc(i); + ln = strdup(o->ln); if (ln == NULL) goto err; - memcpy(ln, o->ln, i); r->ln = ln; } if (o->sn != NULL) { - i = strlen(o->sn) + 1; - sn = malloc(i); + sn = strdup(o->sn); if (sn == NULL) goto err; - memcpy(sn, o->sn, i); r->sn = sn; } r->flags = o->flags | (ASN1_OBJECT_FLAG_DYNAMIC | diff --git a/src/lib/libssl/src/crypto/objects/obj_lib.c b/src/lib/libssl/src/crypto/objects/obj_lib.c index 6dc515e628..247bafbe01 100644 --- a/src/lib/libssl/src/crypto/objects/obj_lib.c +++ b/src/lib/libssl/src/crypto/objects/obj_lib.c @@ -1,4 +1,4 @@ -/* $OpenBSD: obj_lib.c,v 1.12 2014/07/11 08:44:49 jsing Exp $ */ +/* $OpenBSD: obj_lib.c,v 1.13 2014/10/07 04:59:25 miod Exp $ */ /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) * All rights reserved. * @@ -68,7 +68,6 @@ ASN1_OBJECT * OBJ_dup(const ASN1_OBJECT *o) { ASN1_OBJECT *r; - int i; char *ln = NULL, *sn = NULL; unsigned char *data = NULL; @@ -94,20 +93,16 @@ OBJ_dup(const ASN1_OBJECT *o) r->nid = o->nid; r->ln = r->sn = NULL; if (o->ln != NULL) { - i = strlen(o->ln) + 1; - ln = malloc(i); + ln = strdup(o->ln); if (ln == NULL) goto err; - memcpy(ln, o->ln, i); r->ln = ln; } if (o->sn != NULL) { - i = strlen(o->sn) + 1; - sn = malloc(i); + sn = strdup(o->sn); if (sn == NULL) goto err; - memcpy(sn, o->sn, i); r->sn = sn; } r->flags = o->flags | (ASN1_OBJECT_FLAG_DYNAMIC | -- cgit v1.2.3-55-g6feb