Fix SSL_set_tlsext_debug_callback/-tlsextdebug
authorTodd Short <tshort@akamai.com>
Tue, 11 Jul 2017 19:32:10 +0000 (15:32 -0400)
committerRich Salz <rsalz@openssl.org>
Mon, 31 Jul 2017 17:38:05 +0000 (13:38 -0400)
commitb93a295a36efe06fd767c8109220c9df739c2c3a
tree846ead2595ca5975c3f417781902bb3d4758c2e2
parentee1ed1d380f1bd3508186ddf78de678a7072f115
Fix SSL_set_tlsext_debug_callback/-tlsextdebug

Some extensions were being displayed twice, before they were parsed, and
again after they were parsed.
The supported_versions extension was not being fully displayed, as it
was processed differently than other extensions.

Move the debug callback to where the extensions are first collected, to
catch all the extensions as they come in, so they are ordered correctly.

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3911)
ssl/statem/extensions.c