diff options
author | Kazuki Yamaguchi <k@rhe.jp> | 2016-08-03 21:23:15 +0900 |
---|---|---|
committer | Kazuki Yamaguchi <k@rhe.jp> | 2016-08-16 14:22:07 +0900 |
commit | c63e0ed3d76c66758dd6ffeb97ff38414082b65c (patch) | |
tree | 47b2fe103a3c543ef909779d2361950f8d21583c /ext/openssl/ossl_x509revoked.c | |
parent | 31ddb457bfe55171e657ec04d57defdf7d460370 (diff) | |
download | ruby-openssl-c63e0ed3d76c66758dd6ffeb97ff38414082b65c.tar.gz |
x509revoked: fix memory leaks in #set_extensions and #add_extension
X509_REVOKED_add_ext() dups the X509_EXTENSION.
Diffstat (limited to 'ext/openssl/ossl_x509revoked.c')
-rw-r--r-- | ext/openssl/ossl_x509revoked.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ext/openssl/ossl_x509revoked.c b/ext/openssl/ossl_x509revoked.c index 6f170077..e5d59db2 100644 --- a/ext/openssl/ossl_x509revoked.c +++ b/ext/openssl/ossl_x509revoked.c @@ -218,7 +218,7 @@ ossl_x509revoked_set_extensions(VALUE self, VALUE ary) X509_EXTENSION_free(ext); for (i=0; i<RARRAY_LEN(ary); i++) { item = RARRAY_AREF(ary, i); - ext = DupX509ExtPtr(item); + ext = GetX509ExtPtr(item); if(!X509_REVOKED_add_ext(rev, ext, -1)) { ossl_raise(eX509RevError, NULL); } @@ -233,7 +233,7 @@ ossl_x509revoked_add_extension(VALUE self, VALUE ext) X509_REVOKED *rev; GetX509Rev(self, rev); - if(!X509_REVOKED_add_ext(rev, DupX509ExtPtr(ext), -1)) { + if (!X509_REVOKED_add_ext(rev, GetX509ExtPtr(ext), -1)) { ossl_raise(eX509RevError, NULL); } |