Re: [PATCH] MINOR: ssl: free ctx when libssl doesn't support NPN

2018-11-26 Thread Willy Tarreau
On Mon, Nov 26, 2018 at 10:57:17PM +0100, Lukas Tribus wrote: > The previous fix da95fd90 ("BUILD/MINOR: ssl: fix build with non-alpn/ > non-npn libssl") does fix the build in old OpenSSL release, but I > overlooked that the ctx is only freed when NPN is supported. > > Fix this by moving the

[PATCH] MINOR: ssl: free ctx when libssl doesn't support NPN

2018-11-26 Thread Lukas Tribus
The previous fix da95fd90 ("BUILD/MINOR: ssl: fix build with non-alpn/ non-npn libssl") does fix the build in old OpenSSL release, but I overlooked that the ctx is only freed when NPN is supported. Fix this by moving the #endif to the proper place (this was broken in c7566001 ("MINOR: server: Add