[asterisk-commits] bebuild: tag 10.11.0-digiumphones-rc2 r377296 - in /tags/10.11.0-digiumphones...

[SVN commits to the Asterisk project]

Author: bebuild
Date: Wed Dec  5 11:51:47 2012
New Revision: 377296

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=377296
Log:
Merge r377258 for 10.11.0-digiumphones-rc2

Modified:
    tags/10.11.0-digiumphones-rc2/   (props changed)
    tags/10.11.0-digiumphones-rc2/.version
    tags/10.11.0-digiumphones-rc2/ChangeLog
    tags/10.11.0-digiumphones-rc2/channels/chan_sip.c

Propchange: tags/10.11.0-digiumphones-rc2/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Propchange: tags/10.11.0-digiumphones-rc2/
------------------------------------------------------------------------------
    svn:mergeinfo = /branches/10:377258

Modified: tags/10.11.0-digiumphones-rc2/.version
URL: http://svnview.digium.com/svn/asterisk/tags/10.11.0-digiumphones-rc2/.version?view=diff&rev=377296&r1=377295&r2=377296
==============================================================================
--- tags/10.11.0-digiumphones-rc2/.version (original)
+++ tags/10.11.0-digiumphones-rc2/.version Wed Dec  5 11:51:47 2012
@@ -1,1 +1,1 @@
-10.11.0-digiumphones-rc1
+10.11.0-digiumphones-rc2

Modified: tags/10.11.0-digiumphones-rc2/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/tags/10.11.0-digiumphones-rc2/ChangeLog?view=diff&rev=377296&r1=377295&r2=377296
==============================================================================
--- tags/10.11.0-digiumphones-rc2/ChangeLog (original)
+++ tags/10.11.0-digiumphones-rc2/ChangeLog Wed Dec  5 11:51:47 2012
@@ -1,3 +1,23 @@
+2012-12-05  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Asterisk 10.11.0-digiumphones-rc2 Released.
+
+	* Fix a SIP request memory leak with TLS connections.
+
+	During the TLS re-work in chan_sip some TLS specific code was moved
+	into a separate function. This function operates on a copy of the
+	incoming SIP request. This copy was never deinitialized causing a
+	memory leak for each request processed.
+
+	This function is now given a SIP request structure which it can use
+	to copy the incoming request into. This reduces the amount of memory
+	allocations done since the internal allocated components are reused
+	between packets and also ensures the SIP request structure is
+	deinitialized when the TLS connection is torn down.
+
+	(closes issue ASTERISK-20763)
+	Reported by: deti
+
 2012-11-06  Asterisk Development Team <asteriskteam at digium.com>
 
 	* Asterisk 10.11.0-digiumphones-rc1 Released.

Modified: tags/10.11.0-digiumphones-rc2/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/tags/10.11.0-digiumphones-rc2/channels/chan_sip.c?view=diff&rev=377296&r1=377295&r2=377296
==============================================================================
--- tags/10.11.0-digiumphones-rc2/channels/chan_sip.c (original)
+++ tags/10.11.0-digiumphones-rc2/channels/chan_sip.c Wed Dec  5 11:51:47 2012
@@ -2561,10 +2561,10 @@
  * \retval -1 Failed to read data
  * \retval 0 Succeeded in reading data
  */
-static int sip_tls_read(struct sip_request *req, struct ast_tcptls_session_instance *tcptls_session, int authenticated, time_t start, struct sip_threadinfo *me)
+static int sip_tls_read(struct sip_request *req, struct sip_request *reqcpy, struct ast_tcptls_session_instance *tcptls_session,
+			int authenticated, time_t start, struct sip_threadinfo *me)
 {
 	int res, content_length, after_poll = 1, need_poll = 1;
-	struct sip_request reqcpy = { 0, };
 	char buf[1024] = "";
 	int timeout = -1;
 
@@ -2618,10 +2618,10 @@
 		}
 		ast_str_append(&req->data, 0, "%s", buf);
 	}
-	copy_request(&reqcpy, req);
-	parse_request(&reqcpy);
+	copy_request(reqcpy, req);
+	parse_request(reqcpy);
 	/* In order to know how much to read, we need the content-length header */
-	if (sscanf(sip_get_header(&reqcpy, "Content-Length"), "%30d", &content_length)) {
+	if (sscanf(sip_get_header(reqcpy, "Content-Length"), "%30d", &content_length)) {
 		while (content_length > 0) {
 			size_t bytes_read;
 			if (!tcptls_session->client && !authenticated) {
@@ -3035,7 +3035,7 @@
 			req.socket.fd = tcptls_session->fd;
 
 			if (tcptls_session->ssl) {
-				res = sip_tls_read(&req, tcptls_session, authenticated, start, me);
+				res = sip_tls_read(&req, &reqcpy, tcptls_session, authenticated, start, me);
 			} else {
 				res = sip_tcp_read(&req, tcptls_session, authenticated, start);
 			}