From: Benjamin Kaduk Date: Mon, 22 Oct 2018 16:54:20 +0000 (-0500) Subject: Restore sensible "sess_accept" counter tracking X-Git-Tag: openssl-3.0.0-alpha1~2958 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=2aaa0b146b967397a6e61fa8df969e7847f82086;p=oweals%2Fopenssl.git Restore sensible "sess_accept" counter tracking Commit 9ef9088c1585e13b9727796f15f77da64dbbe623 switched the SSL/SSL_CTX statistics counters to using Thread-Sanitizer-friendly primitives. However, it erroneously converted an addition of -1 (for s->session_ctx->stats.sess_accept) to an addition of +1, since that is the only counter API provided by the internal tsan_assist.h header until the previous commit. This means that for each accepted (initial) connection, the session_ctx's counter would get doubly incremented, and the (switched) ctx's counter would also get incremented. Restore the counter decrement so that each accepted connection increments exactly one counter exactly once (in net effect). Reviewed-by: Paul Dale (Merged from https://github.com/openssl/openssl/pull/7464) --- diff --git a/ssl/statem/extensions.c b/ssl/statem/extensions.c index 8d4939d601..ad4256d370 100644 --- a/ssl/statem/extensions.c +++ b/ssl/statem/extensions.c @@ -962,7 +962,7 @@ static int final_server_name(SSL *s, unsigned int context, int sent) */ if (SSL_IS_FIRST_HANDSHAKE(s) && s->ctx != s->session_ctx) { tsan_counter(&s->ctx->stats.sess_accept); - tsan_counter(&s->session_ctx->stats.sess_accept); + tsan_decr(&s->session_ctx->stats.sess_accept); } /*