From: Mark Michelson <mmichelson@digium.com>
Date: Thu, 25 Mar 2010 17:12:25 +0000 (+0000)
Subject: Fix potential crashes from trying to reference nonexistent RTP streams.
X-Git-Tag: 1.6.2.7-rc1~20
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=70d5a1c27dfaad1b4ffb491339737963e875e41b;p=thirdparty%2Fasterisk.git

Fix potential crashes from trying to reference nonexistent RTP streams.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.2@254542 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 31c910a886..0c946cf733 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -20957,19 +20957,26 @@ static int acf_channel_read(struct ast_channel *chan, const char *funcname, char
 		ast_copy_string(buf, (p->t38.state == T38_DISABLED) ? "0" : "1", buflen);
 	} else if (!strcasecmp(args.param, "rtpdest")) {
 		struct sockaddr_in sin;
+		struct ast_rtp *stream;
 
 		if (ast_strlen_zero(args.type))
 			args.type = "audio";
 
-		if (!strcasecmp(args.type, "audio"))
-			ast_rtp_get_peer(p->rtp, &sin);
-		else if (!strcasecmp(args.type, "video"))
-			ast_rtp_get_peer(p->vrtp, &sin);
-		else if (!strcasecmp(args.type, "text"))
-			ast_rtp_get_peer(p->trtp, &sin);
-		else
+		if (!strcasecmp(args.type, "audio")) {
+			stream = p->rtp;
+		} else if (!strcasecmp(args.type, "video")) {
+			stream = p->vrtp;
+		} else if (!strcasecmp(args.type, "text")) {
+			stream = p->trtp;
+		} else {
+			return -1;
+		}
+
+		if (!stream) {
 			return -1;
+		}
 
+		ast_rtp_get_peer(stream, &sin);
 		snprintf(buf, buflen, "%s:%d", ast_inet_ntoa(sin.sin_addr), ntohs(sin.sin_port));
 	} else if (!strcasecmp(args.param, "rtpqos")) {
 		struct ast_rtp_quality qos;