diff options
author | tenderlove <tenderlove@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-07-22 19:04:58 +0000 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2015-07-22 14:30:41 -0700 |
commit | 006784895052e030bdcfade658b65f9414594dbc (patch) | |
tree | a64a137eb34c8be9dbc083d5337055fae25fa285 /ext | |
parent | dedd99217989dff96a4b9c71ba7bc3820f30b0ce (diff) | |
download | ruby-openssl-006784895052e030bdcfade658b65f9414594dbc.tar.gz |
fix tests bu not setting the string instance on the frozen object
OpenSSL [copies the string returned by the pointe](https://github.com/openssl/openssl/blob/9f040d6decca7930e978784c917f731e5c45e8f0/ssl/t1_lib.c#L1800-1809), so it should be safe to just return a pointer to the string object and not set an instance variable on the already frozen object.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@51349 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'ext')
-rw-r--r-- | ext/openssl/ossl_ssl.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ext/openssl/ossl_ssl.c b/ext/openssl/ossl_ssl.c index 04b115e2..f1b2f46c 100644 --- a/ext/openssl/ossl_ssl.c +++ b/ext/openssl/ossl_ssl.c @@ -707,7 +707,6 @@ ssl_alpn_select_cb(SSL *ssl, const unsigned char **out, unsigned char *outlen, c selected = rb_funcall(cb, rb_intern("call"), 1, protocols); *out = (unsigned char *) StringValuePtr(selected); *outlen = RSTRING_LENINT(selected); - rb_iv_set(sslctx_obj, "@_alpn_selected", selected); return SSL_TLSEXT_ERR_OK; } |