Fix call to SSL_SESSION_up_ref().
authorZonr Chang <zonr.net@gmail.com>
Wed, 12 Oct 2016 20:14:31 +0000 (13:14 -0700)
committerFacebook Github Bot <facebook-github-bot-bot@fb.com>
Wed, 12 Oct 2016 20:23:53 +0000 (13:23 -0700)
Summary:
`SSL_SESSION_up_ref()` in both BoringSSL and OpenSSL 1.1.0 takes a
`SSL_SESSION*`.
Closes https://github.com/facebook/folly/pull/493

Reviewed By: yfeldblum

Differential Revision: D4009706

Pulled By: Orvid

fbshipit-source-id: 68ea201821e7a6d5835a79b34187344ba251e239

folly/ssl/detail/SSLSessionImpl.cpp

index 7a83c1e76dc2c99cd7a5bdfa3dfa60d5c0e58790..d93fcaf96cafc9ca2016e75f891b50b17e266425 100644 (file)
@@ -99,7 +99,7 @@ void SSLSessionImpl::upRef() {
 #if defined(OPENSSL_IS_102) || defined(OPENSSL_IS_101)
     CRYPTO_add(&session_->references, 1, CRYPTO_LOCK_SSL_SESSION);
 #elif defined(OPENSSL_IS_BORINGSSL) || defined(OPENSSL_IS_110)
-    SSL_SESSION_up_ref(&session_);
+    SSL_SESSION_up_ref(session_);
 #endif
   }
 }