[Vortex] PATCH: Fix TLS accept

Francis Brosnan Blazquez francis at aspl.es
Fri Mar 7 16:58:02 CET 2008


Hi Jens,

> I finally updated my sources to the latest revision in SVN (3316) and  
> in doing so found a bug-fix I must not have submitted yet.
> 
> The bug is in the function vortex_tls_process_start_msg: it checks  
> whether cert and key files are registered, and causes the TLS  
> negotiation to fail if there aren't any. But this is wrong, because  
> the files aren't needed if a client has registered a  
> VortexTlsCtxCreation callback instead.
> 
> My patch conditionalizes this code to run only if both ctx- 
>  >tls_default_ctx_creation and vortex_connection_get_data (connection,  
> CTX_CREATION) are NULL. Without the fix, TLS connections always fail  
> in my app; with it, they work.

Nice Jens. I've applied the patch with some minimal modifications (i.e,
variables must be declared before any sentence ;-). 

Thanks for reporting and fixing. Cheers!

> —Jens
> 
> _______________________________________________
> Vortex mailing list
> Vortex at lists.aspl.es
> http://lists.aspl.es/cgi-bin/mailman/listinfo/vortex
-- 
Francis Brosnan Blazquez <francis at aspl.es>
Advanced Software Production Line, S.L.




More information about the Vortex mailing list