From 10c2ea3ca298d3e9fe50b64fd466edbe1fbf1b61 Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Thu, 12 Apr 2018 18:11:45 -0400 Subject: [PATCH] Do not reach into BoringSSL internals. SSL_SESSION is a private struct and should not be accessed by calling code. There is no need to assert on the reference count in that test; the test already asserts on whether the SSL_SESSION was destroyed. --- test/core/tsi/ssl_session_cache_test.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/test/core/tsi/ssl_session_cache_test.cc b/test/core/tsi/ssl_session_cache_test.cc index 72df0e545c3..c86cefb3ff4 100644 --- a/test/core/tsi/ssl_session_cache_test.cc +++ b/test/core/tsi/ssl_session_cache_test.cc @@ -88,7 +88,6 @@ TEST(SslSessionCacheTest, InitialState) { // Verify session initial state. { tsi::SslSessionPtr tmp_sess = tracker.NewSession(1); - EXPECT_EQ(tmp_sess->references, 1); EXPECT_TRUE(tracker.IsAlive(1)); EXPECT_EQ(tracker.AliveCount(), 1); }