[Asterisk-code-review] tcptls: Do not re-bind to wildcard on client creation. (asterisk[13])

Alexander Traud asteriskteam at digium.com
Sun Oct 8 09:22:58 CDT 2017


Alexander Traud has uploaded this change for review. ( https://gerrit.asterisk.org/6681


Change subject: tcptls: Do not re-bind to wildcard on client creation.
......................................................................

tcptls: Do not re-bind to wildcard on client creation.

Since ASTERISK-26922, this issue affected only those chan_sip which were
* enabled for dual-stack (bindaddr=::), and
* enabled for TCP (tcpenable=yes) and/or TLS (tlsenable=yes), and
* tried to register and/or invite a IPv4-only service,
* via TCP and/or TLS.
Now, ast_tcptls_client_create does not re-bind to [::] anymore.

ASTERISK-27324 #close

Change-Id: I4b242837bdeb1ec7130dc82505c6180a946fd9b5
---
M main/tcptls.c
1 file changed, 2 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/81/6681/1

diff --git a/main/tcptls.c b/main/tcptls.c
index a34b41e..820a03e 100644
--- a/main/tcptls.c
+++ b/main/tcptls.c
@@ -1119,7 +1119,8 @@
 
 	/* if a local address was specified, bind to it so the connection will
 	   originate from the desired address */
-	if (!ast_sockaddr_isnull(&desc->local_address)) {
+	if (!ast_sockaddr_isnull(&desc->local_address) &&
+	    !ast_sockaddr_is_any(&desc->local_address)) {
 		setsockopt(desc->accept_fd, SOL_SOCKET, SO_REUSEADDR, &x, sizeof(x));
 		if (ast_bind(desc->accept_fd, &desc->local_address)) {
 			ast_log(LOG_ERROR, "Unable to bind %s to %s: %s\n",

-- 
To view, visit https://gerrit.asterisk.org/6681
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4b242837bdeb1ec7130dc82505c6180a946fd9b5
Gerrit-Change-Number: 6681
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171008/594b6269/attachment.html>


More information about the asterisk-code-review mailing list