[omniORB] Patch to fix leak in SSL context initialization

Peter Klotz peter.klotz at aon.at
Sun Oct 18 17:56:53 BST 2009


Peter Klotz wrote:
> Hello
> 
> Initializing and destroying an SSL enabled omniORB in a loop results in 
> memory leaks because the SSL context string parameters are allocated but 
> never freed.
> 
> These 3 lines in sslTransportImpl.cc cause the problem:
> 
>      sslContext::certificate_authority_file = CORBA::string_dup(value);
>      sslContext::key_file = CORBA::string_dup(value);
>      sslContext::key_file_password = CORBA::string_dup(value);
> 
> The attached patch against omniORB 4.1.4 simply calls 
> CORBA::string_free() before allocating the new value.
> 
> Since all values are zero initialized on startup the fix should be safe.

Hello Duncan

Could you please apply this patch or do you have any objections?

Regards, Peter.



More information about the omniORB-list mailing list