BUG/MAJOR: ssl: fix a regression in ssl_sock_shutw()
authorWilly Tarreau <w@1wt.eu>
Mon, 13 Feb 2017 10:12:29 +0000 (11:12 +0100)
committerWilly Tarreau <w@1wt.eu>
Mon, 13 Feb 2017 10:15:13 +0000 (11:15 +0100)
commite3cc3a302651e905209b96e07203d03cf7e46ccb
tree1c54c20117b6806f1e508109bb08e8ce89a87a3a
parent62c8a21c10c41629d63e93bee904514f456def74
BUG/MAJOR: ssl: fix a regression in ssl_sock_shutw()

Commit 405ff31 ("BUG/MINOR: ssl: assert on SSL_set_shutdown with BoringSSL")
introduced a regression causing some random crashes apparently due to
memory corruption. The issue is the use of SSL_CTX_set_quiet_shutdown()
instead of SSL_set_quiet_shutdown(), making it use a different structure
and causing the flag to be put who-knows-where.

Many thanks to Jarno Huuskonen who reported this bug early and who
bisected the issue to spot this patch. No backport is needed, this
is 1.8-specific.
src/ssl_sock.c