From cdabd6d89101b4aa090a0f858b81947548abbee6 Mon Sep 17 00:00:00 2001 From: Ted Unangst Date: Wed, 7 May 2014 21:18:54 +0000 Subject: don't free memory unless we allocated it. reported to openssl by David Ramos (rt 3338) stealing commit from logan, who's run away. ok miod. --- lib/libcrypto/ec/ec_asn1.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'lib/libcrypto') diff --git a/lib/libcrypto/ec/ec_asn1.c b/lib/libcrypto/ec/ec_asn1.c index 70670a4c750..0ea0d6e2a43 100644 --- a/lib/libcrypto/ec/ec_asn1.c +++ b/lib/libcrypto/ec/ec_asn1.c @@ -1233,8 +1233,10 @@ i2o_ECPublicKey(EC_KEY * a, unsigned char **out) if (!EC_POINT_point2oct(a->group, a->pub_key, a->conv_form, *out, buf_len, NULL)) { ECerr(EC_F_I2O_ECPUBLICKEY, ERR_R_EC_LIB); - free(*out); - *out = NULL; + if (new_buffer) { + free(*out); + *out = NULL; + } return 0; } if (!new_buffer) -- cgit v1.2.3