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

[Paul Belanger]

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

-- 
Paul Belanger | PolyBeacon, Inc.
Jabber: paul.belanger at polybeacon.com | IRC: pabelanger (Freenode)
Github: https://github.com/pabelanger | Twitter: 
https://twitter.com/pabelanger