[Asterisk-code-review] http: leverage 'bindaddr' for TLS in http.conf (asterisk[master])

Alexander Traud asteriskteam at digium.com
Mon Jun 20 08:07:46 CDT 2016


Alexander Traud has uploaded a new change for review.

  https://gerrit.asterisk.org/3041

Change subject: http: leverage 'bindaddr' for TLS in http.conf
......................................................................

http: leverage 'bindaddr' for TLS in http.conf

The internal HTTP/WebSocket server supports both TCP and TLS, which can be
activated separately via the file http.conf. The source code intends to re-use
the TCP parameter 'bindaddr' for TLS, even if 'tlsbindaddr' is not specified
explicitly. This did not work because of a typo. This change resolves this typo.

ASTERISK-26126 #close

Change-Id: I5efb0409ae12044dfb3495b6b97b6d40a8c9c51f
---
M main/http.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/41/3041/1

diff --git a/main/http.c b/main/http.c
index a836282..bc23e6e 100644
--- a/main/http.c
+++ b/main/http.c
@@ -2220,7 +2220,7 @@
 		 * the non-TLS bindaddress here.
 		 */
 		if (ast_sockaddr_isnull(&https_desc.local_address) && http_desc.accept_fd != -1) {
-			ast_sockaddr_copy(&https_desc.local_address, &https_desc.local_address);
+			ast_sockaddr_copy(&https_desc.local_address, &http_desc.local_address);
 			/* Of course, we can't use the same port though.
 			 * Since no bind address was specified, we just use the
 			 * default TLS port

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5efb0409ae12044dfb3495b6b97b6d40a8c9c51f
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>



More information about the asterisk-code-review mailing list