Provide SSL_CTX.stats.sess_accept for switched ctxs
authorBenjamin Kaduk <bkaduk@akamai.com>
Wed, 18 Oct 2017 13:05:57 +0000 (08:05 -0500)
committerBen Kaduk <kaduk@mit.edu>
Mon, 30 Oct 2017 15:21:10 +0000 (10:21 -0500)
commit3be08e301146fc449315b7061d086edddb186850
tree544dba59e70345aa440220116023eff47ae63bcd
parent0e6161bcae451ed17346c51db1ddc61fea5f3ec2
Provide SSL_CTX.stats.sess_accept for switched ctxs

We currently increment the SSL_CTX stats.sess_accept field in
tls_setup_handshake(), which is invoked from the state machine well
before ClientHello processing would have had a chance to switch
the SSL_CTX attached to the SSL object due to a provided SNI value.
However, stats.sess_accept_good is incremented in tls_finish_handshake(),
and uses the s->ctx.stats field (i.e., the new SSL_CTX that was switched
to as a result of SNI processing).  This leads to the confusing
(nonsensical) situation where stats.sess_accept_good is larger than
stats.sess_accept, as the "sess_accept" value was counted on the
s->session_ctx.

In order to provide some more useful numbers, increment
s->ctx.stats.sess_accept after SNI processing if the SNI processing
changed s->ctx to differ from s->session_ctx.  To preserve the
property that any given accept is counted only once, make the
corresponding decrement to s->session_ctx.stats.sess_accept when
doing so.

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/4549)
ssl/statem/extensions.c