[asterisk-commits] bebuild: tag 12.7.2 r429307 - in /tags/12.7.2: ./ channels/ res/

[SVN commits to the Asterisk project]

Author: bebuild
Date: Wed Dec 10 08:27:55 2014
New Revision: 429307

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=429307
Log:
Merge r429272 for AST-2014-019

Modified:
    tags/12.7.2/   (props changed)
    tags/12.7.2/ChangeLog
    tags/12.7.2/channels/chan_sip.c
    tags/12.7.2/res/res_http_websocket.c
    tags/12.7.2/res/res_pjsip_transport_websocket.c

Propchange: tags/12.7.2/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Wed Dec 10 08:27:55 2014
@@ -1,1 +1,1 @@
-/branches/12:427382,428301,428304,428333-428334,428409,428422
+/branches/12:427382,428301,428304,428333-428334,428409,428422,429272

Modified: tags/12.7.2/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/tags/12.7.2/ChangeLog?view=diff&rev=429307&r1=429306&r2=429307
==============================================================================
--- tags/12.7.2/ChangeLog (original)
+++ tags/12.7.2/ChangeLog Wed Dec 10 08:27:55 2014
@@ -1,3 +1,25 @@
+2014-12-10  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Asterisk 12.7.2 Released.
+
+	* AST-2014-019: Fix crash when receiving a WebSocket packet with a
+	  payload length of zero.
+
+	  Frames with a payload length of 0 were incorrectly handled in
+	  res_http_websocket. Provided a frame with a payload had been
+	  received prior it was possible for a double free to occur. The
+	  realloc operation would succeed (thus freeing the payload) but be
+	  treated as an error. When the session was then torn down the payload
+	  would be freed again causing a crash. The read function now takes
+	  this into account.
+
+	  This change also fixes assumptions made by users of
+	  res_http_websocket. There is no guarantee that a frame received from
+	  it will be NULL terminated.
+
+	  ASTERISK-24472 #close
+	  Reported by: Badalian Vyacheslav
+
 2014-11-20  Asterisk Development Team <asteriskteam at digium.com>
 
 	* Asterisk 12.7.1 Released.

Modified: tags/12.7.2/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/tags/12.7.2/channels/chan_sip.c?view=diff&rev=429307&r1=429306&r2=429307
==============================================================================
--- tags/12.7.2/channels/chan_sip.c (original)
+++ tags/12.7.2/channels/chan_sip.c Wed Dec 10 08:27:55 2014
@@ -2685,12 +2685,16 @@
 
 		if (opcode == AST_WEBSOCKET_OPCODE_TEXT || opcode == AST_WEBSOCKET_OPCODE_BINARY) {
 			struct sip_request req = { 0, };
+			char data[payload_len + 1];
 
 			if (!(req.data = ast_str_create(payload_len + 1))) {
 				goto end;
 			}
 
-			if (ast_str_set(&req.data, -1, "%s", payload) == AST_DYNSTR_BUILD_FAILED) {
+			strncpy(data, payload, payload_len);
+			data[payload_len] = '\0';
+
+			if (ast_str_set(&req.data, -1, "%s", data) == AST_DYNSTR_BUILD_FAILED) {
 				deinit_req(&req);
 				goto end;
 			}

Modified: tags/12.7.2/res/res_http_websocket.c
URL: http://svnview.digium.com/svn/asterisk/tags/12.7.2/res/res_http_websocket.c?view=diff&rev=429307&r1=429306&r2=429307
==============================================================================
--- tags/12.7.2/res/res_http_websocket.c (original)
+++ tags/12.7.2/res/res_http_websocket.c Wed Dec 10 08:27:55 2014
@@ -508,14 +508,6 @@
 			}
 		}
 
-		if (!(new_payload = ast_realloc(session->payload, (session->payload_len + *payload_len)))) {
-			ast_log(LOG_WARNING, "Failed allocation: %p, %zu, %"PRIu64"\n",
-				session->payload, session->payload_len, *payload_len);
-			*payload_len = 0;
-			ast_websocket_close(session, 1009);
-			return 0;
-		}
-
 		/* Per the RFC for PING we need to send back an opcode with the application data as received */
 		if ((*opcode == AST_WEBSOCKET_OPCODE_PING) && (ast_websocket_write(session, AST_WEBSOCKET_OPCODE_PONG, *payload, *payload_len))) {
 			*payload_len = 0;
@@ -523,9 +515,22 @@
 			return 0;
 		}
 
-		session->payload = new_payload;
-		memcpy((session->payload + session->payload_len), (*payload), (*payload_len));
-		session->payload_len += *payload_len;
+		if (*payload_len) {
+			if (!(new_payload = ast_realloc(session->payload, (session->payload_len + *payload_len)))) {
+				ast_log(LOG_WARNING, "Failed allocation: %p, %zu, %"PRIu64"\n",
+					session->payload, session->payload_len, *payload_len);
+				*payload_len = 0;
+				ast_websocket_close(session, 1009);
+				return 0;
+			}
+
+			session->payload = new_payload;
+			memcpy((session->payload + session->payload_len), (*payload), (*payload_len));
+			session->payload_len += *payload_len;
+		} else if (!session->payload_len && session->payload) {
+			ast_free(session->payload);
+			session->payload = NULL;
+		}
 
 		if (!fin && session->reconstruct && (session->payload_len < session->reconstruct)) {
 			/* If this is not a final message we need to defer returning it until later */

Modified: tags/12.7.2/res/res_pjsip_transport_websocket.c
URL: http://svnview.digium.com/svn/asterisk/tags/12.7.2/res/res_pjsip_transport_websocket.c?view=diff&rev=429307&r1=429306&r2=429307
==============================================================================
--- tags/12.7.2/res/res_pjsip_transport_websocket.c (original)
+++ tags/12.7.2/res/res_pjsip_transport_websocket.c Wed Dec 10 08:27:55 2014
@@ -200,7 +200,8 @@
 
 	pj_gettimeofday(&rdata->pkt_info.timestamp);
 
-	pj_memcpy(rdata->pkt_info.packet, read_data->payload, sizeof(rdata->pkt_info.packet));
+	pj_memcpy(rdata->pkt_info.packet, read_data->payload,
+		PJSIP_MAX_PKT_LEN < read_data->payload_len ? PJSIP_MAX_PKT_LEN : read_data->payload_len);
 	rdata->pkt_info.len = read_data->payload_len;
 	rdata->pkt_info.zero = 0;