diff options
author | Matthew Jordan <mjordan@digium.com> | 2014-10-06 00:53:37 +0000 |
---|---|---|
committer | Matthew Jordan <mjordan@digium.com> | 2014-10-06 00:53:37 +0000 |
commit | c01391686997f8d7c40bfddfe95efdb4cec130b5 (patch) | |
tree | a72ac0f1cdfa5c69c726016df6cb387cbae5dd81 /channels/pjsip | |
parent | 45b7b474aca2a4213a0f4ef06debb17a30d60b2d (diff) |
pjsip/dialplan_functions: Handle PJSIP_MEDIA_OFFER called on non-PJSIP channels
Calling PJSIP_MEDIA_OFFER on a non-PJSIP channel is hazardous to your health.
It will treat the channels as a PJSIP channel, eventually hitting an ao2 error,
FRACKing on assertion error, and quite likely crashing.
This patch adds checks to the read/write callbacks that ensure that the channel
technology is of type 'PJSIP' before attempting to operate on the channel.
#SIPit31
ASTERISK-24382 #close
Reported by: Matt Jordan
........
Merged revisions 424621 from http://svn.asterisk.org/svn/asterisk/branches/12
........
Merged revisions 424622 from http://svn.asterisk.org/svn/asterisk/branches/13
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@424623 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'channels/pjsip')
-rw-r--r-- | channels/pjsip/dialplan_functions.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/channels/pjsip/dialplan_functions.c b/channels/pjsip/dialplan_functions.c index 054552b5a..6c0aff30b 100644 --- a/channels/pjsip/dialplan_functions.c +++ b/channels/pjsip/dialplan_functions.c @@ -707,7 +707,7 @@ int pjsip_acf_channel_read(struct ast_channel *chan, const char *cmd, char *data /* Sanity check */ if (strcmp(ast_channel_tech(chan)->type, "PJSIP")) { - ast_log(LOG_ERROR, "Cannot call %s on a non-PJSIP channel\n", cmd); + ast_log(LOG_WARNING, "Cannot call %s on a non-PJSIP channel\n", cmd); return 0; } @@ -866,6 +866,11 @@ int pjsip_acf_media_offer_read(struct ast_channel *chan, const char *cmd, char * return -1; } + if (strcmp(ast_channel_tech(chan)->type, "PJSIP")) { + ast_log(LOG_WARNING, "Cannot call %s on a non-PJSIP channel\n", cmd); + return -1; + } + channel = ast_channel_tech_pvt(chan); if (!strcmp(data, "audio")) { @@ -889,6 +894,11 @@ int pjsip_acf_media_offer_write(struct ast_channel *chan, const char *cmd, char return -1; } + if (strcmp(ast_channel_tech(chan)->type, "PJSIP")) { + ast_log(LOG_WARNING, "Cannot call %s on a non-PJSIP channel\n", cmd); + return -1; + } + channel = ast_channel_tech_pvt(chan); mdata.session = channel->session; |