[asterisk-dev] [svn-commits] mmichelson: branch 1.8 r367002 - in /branches/1.8: channels/ include/asterisk...

[Mark Michelson]

On 05/23/2012 02:25 PM, Paul Belanger wrote:
> On 12-05-18 12:54 PM, SVN commits to the Digium repositories wrote:
>> Author: mmichelson
>> Date: Fri May 18 11:53:47 2012
>> New Revision: 367002
>>
>> URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=367002
>> Log:
>> Fix memory leak of SSL_CTX structures in TLS core.
>>
>> SSL_CTX structures were allocated but never freed. This was a bigger
>> issue for clients than servers since new SSL_CTX structures could be
>> allocated for each connection. Servers, on the other hand, typically
>> set up a single SSL_CTX for their lifetime.
>>
>> This is solved in two ways:
>>
>> 1. In __ssl_setup(), if a tcptls_cfg has an ssl_ctx on it, it is
>> freed so that a new one can take its place.
>> 2. A companion to ast_ssl_setup() called ast_ssl_teardown() has
>> been added so that servers can properly free their SSL_CTXs.
>>
>> (issue ASTERISK-19278)
>>
> You may have to revisit this commit, it break compiling on Ubuntu 
> 10.04, see below.
>
> ---
> tcptls.o: In function `ast_ssl_teardown':
> /home/pabelanger/svn/digium/asterisk/testing/1.8/main/tcptls.c:407: 
> undefined reference to `SSL_CTX_free'
> /home/pabelanger/svn/digium/asterisk/testing/1.8/main/tcptls.c:407: 
> undefined reference to `SSL_CTX_free'
> collect2: ld returned 1 exit status
> make[1]: *** [asterisk] Error 1
> make: *** [main] Error 2
>
Thanks for pointing this out. Try revision 367416 and see if it's fixed.

Mark!