From: Tilghman Lesher Date: Mon, 18 Jun 2007 19:52:56 +0000 (+0000) Subject: Merged revisions 69796 via svnmerge from X-Git-Tag: 1.6.0-beta1~3^2~2345 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a67890d7a95b1b248a6930d2c4ce2c1df6e4a031;p=thirdparty%2Fasterisk.git Merged revisions 69796 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r69796 | tilghman | 2007-06-18 14:48:17 -0500 (Mon, 18 Jun 2007) | 2 lines Issue 10005 - Segfault with missing arguments, plus fix a missing define for SIP INFO channels ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@69797 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 6b4834d906..991695d300 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -1646,6 +1646,7 @@ static const struct ast_channel_tech sip_tech_info = { .send_digit_end = sip_senddigit_end, .bridge = ast_rtp_bridge, .send_text = sip_sendtext, + .func_channel_read = acf_channel_read, }; /**--- some list management macros. **/ @@ -14992,12 +14993,15 @@ static int acf_channel_read(struct ast_channel *chan, const char *funcname, char if (ast_strlen_zero(args.param) || strcasecmp(args.param, "rtpqos")) return -1; + /* Default arguments of audio,all */ + if (ast_strlen_zero(args.type)) + args.type = "audio"; + if (ast_strlen_zero(args.field)) + args.field = "all"; + memset(buf, 0, buflen); memset(&qos, 0, sizeof(qos)); - if (ast_strlen_zero(args.type)) - return -1; - if (strcasecmp(args.type, "AUDIO") == 0) { all = ast_rtp_get_quality(p->rtp, &qos); } else if (strcasecmp(args.type, "VIDEO") == 0) { @@ -15006,9 +15010,6 @@ static int acf_channel_read(struct ast_channel *chan, const char *funcname, char all = ast_rtp_get_quality(p->trtp, &qos); } - if (ast_strlen_zero(args.field)) - return -1; - if (strcasecmp(args.field, "local_ssrc") == 0) snprintf(buf, buflen, "%u", qos.local_ssrc); else if (strcasecmp(args.field, "local_lostpackets") == 0)