diff options
| author | David Benjamin <davidben@chromium.org> | 2015-02-09 19:56:18 -0500 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2015-02-10 01:23:51 +0000 |
| commit | ed3d302190d06f0fa27f9455297ec70edf804587 (patch) | |
| tree | 4692eff465119d12d7ba9a5e8e174959522ef2a2 | |
| parent | 9e128b06a110044638e793b197744a79f6aa4747 (diff) | |
| download | boringssl-ed3d302190d06f0fa27f9455297ec70edf804587.zip boringssl-ed3d302190d06f0fa27f9455297ec70edf804587.tar.gz boringssl-ed3d302190d06f0fa27f9455297ec70edf804587.tar.bz2 | |
Unrefcount SSL*.
Nothing ever increments the count.
Change-Id: I0b9396e0f5755fa7d9cfd522e17910c760c1aebd
Reviewed-on: https://boringssl-review.googlesource.com/3360
Reviewed-by: Adam Langley <agl@google.com>
| -rw-r--r-- | include/openssl/ssl.h | 1 | ||||
| -rw-r--r-- | ssl/ssl_lib.c | 9 |
2 files changed, 0 insertions, 10 deletions
diff --git a/include/openssl/ssl.h b/include/openssl/ssl.h index c41ed73..19f9f00 100644 --- a/include/openssl/ssl.h +++ b/include/openssl/ssl.h @@ -1262,7 +1262,6 @@ struct ssl_st { /* for server side, keep the list of CA_dn we can use */ STACK_OF(X509_NAME) *client_CA; - int references; unsigned long options; /* protocol behaviour */ unsigned long mode; /* API behaviour */ long max_cert_list; diff --git a/ssl/ssl_lib.c b/ssl/ssl_lib.c index 75a36d7..34fff17 100644 --- a/ssl/ssl_lib.c +++ b/ssl/ssl_lib.c @@ -345,8 +345,6 @@ SSL *SSL_new(SSL_CTX *ctx) { s->enc_method = ssl3_get_enc_method(s->version); assert(s->enc_method != NULL); - s->references = 1; - s->rwstate = SSL_NOTHING; s->rstate = SSL_ST_READ_HEADER; @@ -547,17 +545,10 @@ X509_VERIFY_PARAM *SSL_get0_param(SSL *ssl) { return ssl->param; } void SSL_certs_clear(SSL *s) { ssl_cert_clear_certs(s->cert); } void SSL_free(SSL *s) { - int i; - if (s == NULL) { return; } - i = CRYPTO_add(&s->references, -1, CRYPTO_LOCK_SSL); - if (i > 0) { - return; - } - if (s->param) { X509_VERIFY_PARAM_free(s->param); } |