Branch: refs/heads/3.5
Home: https://github.com/OpenSIPS/opensips
Commit: 0485ed1bf548f32f5a89d4f1f0b212344787d5f1
https://github.com/OpenSIPS/opensips/commit/0485ed1bf548f32f5a89d4f1f0b212344787d5f1
Author: Your Name <your-em...@example.com>
Date: 2025-04-17 (Thu, 17 Apr 2025)
Changed paths:
M modules/tls_openssl/openssl_conn_ops.c
Log Message:
-----------
tls_openssl: fix openssl_tls_async_connect
The `fd` value here is used with `poll()`/`select()` in the event that
`SSL_connect()`
gives an `SSL_ERROR_WANT_WRITE` error. So we need to be polling/selecting on
the fd that
is used by the underlying socket, which is the fd passed as an argument to the
function,
rather than `con->fd`.
It seems to me that `handle_io()` updates `con->fd` to match the received fd
number only
when the socket wants reading and not when it wants writing. When the socket
wants writing
the correct fd to use is the one that is passed in the function argument, not
the one
that is in `con->fd`.
The reasons this mostly worked before are:
1. on a box that is not heavily-loaded you often end up with the same worker
setting up
the TLS session as opened the socket in the first place, so the fd in
`con->fd happens
to be correct.
2. if `SSL_connect()` happily connects straight away, without an
`SSL_ERROR_WANT_WRITE`,
then you never run into a code path that uses the value in `fd` anyway.
(cherry picked from commit 58977a99484120d32e1bf7968a54c140e6465e78)
To unsubscribe from these emails, change your notification settings at
https://github.com/OpenSIPS/opensips/settings/notifications
_______________________________________________
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
