Revert "BUG/MINOR: ssl: fix potential memory leak in ssl_sock_load_dh_params()"
authorWilly Tarreau <w@1wt.eu>
Thu, 30 Jun 2016 18:00:19 +0000 (20:00 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 30 Jun 2016 18:00:19 +0000 (20:00 +0200)
commit90fd35c3a726e613e36ea0399507778b094181a0
treee801df74f836740875882d6406b7b8e4c09e4684
parent831962e3b32777573f3818c0917e7592bf3b462f
Revert "BUG/MINOR: ssl: fix potential memory leak in ssl_sock_load_dh_params()"

This reverts commit 0ea4c23ca754c3e6c005b67403a0619ca17d4587.

Certain very simple confs randomly segfault upon startup with openssl 1.0.2
with this patch, which seems to indicate a use after free. Better drop it
and let valgrind complain about the potential leak.

Also it's worth noting that the man page for SSL_CTX_set_tmp_dh() makes no
mention about whether or not the element should be freed, and the example
provided does not use it either.

This fix should be backported to 1.6 and 1.5 where the patch was just
included.
src/ssl_sock.c