[omniORB] Patch to fix leak in SSL context initialization

[Peter Klotz]

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.