diff options
author | Joshua Colp <jcolp@digium.com> | 2012-09-25 20:14:13 +0000 |
---|---|---|
committer | Joshua Colp <jcolp@digium.com> | 2012-09-25 20:14:13 +0000 |
commit | 318c7bea44516d010b0b544dc3c632b746657d83 (patch) | |
tree | 2299e0c2870d2e34cbfe3fe6aaef1929fa01360d /channels | |
parent | fdfb3ae5faac680bf939eea1312919bee14db76f (diff) |
Fix T.38 support when used with chan_local in between.
Users of the T.38 API can indicate AST_T38_REQUEST_PARMS on a channel to request that the
channel indicate a T.38 negotiation with the parameters present on the channel. The return
value of this indication is expected to be AST_T38_REQUEST_PARMS upon success but with
chan_local involved this could never occur.
This fix changes chan_local to always return AST_T38_REQUEST_PARMS for this situation. If
the underlying channel technology on the other side does not support T.38 this would have
been determined ahead of time using ast_channel_get_t38_state and an indication would
not occur.
(closes issue ASTERISK-20229)
Reported by: wdoekes
Patches:
ASTERISK-20229.patch uploaded by wdoekes (license 5674)
Review: https://reviewboard.asterisk.org/r/2070/
........
Merged revisions 373705 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 373706 from http://svn.asterisk.org/svn/asterisk/branches/10
........
Merged revisions 373707 from http://svn.asterisk.org/svn/asterisk/branches/11
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@373708 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'channels')
-rw-r--r-- | channels/chan_local.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/channels/chan_local.c b/channels/chan_local.c index c9740cd27..fcef1c7f1 100644 --- a/channels/chan_local.c +++ b/channels/chan_local.c @@ -757,6 +757,15 @@ static int local_indicate(struct ast_channel *ast, int condition, const void *da f.data.ptr = (void *) data; f.datalen = datalen; res = local_queue_frame(p, isoutbound, &f, ast, 1); + + if (!res && (condition == AST_CONTROL_T38_PARAMETERS) && + (datalen == sizeof(struct ast_control_t38_parameters))) { + const struct ast_control_t38_parameters *parameters = data; + + if (parameters->request_response == AST_T38_REQUEST_PARMS) { + res = AST_T38_REQUEST_PARMS; + } + } } else { ast_debug(4, "Blocked indication %d\n", condition); } |