[asterisk-commits] bebuild: tag 1.8.17.0-rc2 r374294 - in /tags/1.8.17.0-rc2: ./ channels/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Wed Oct 3 09:15:28 CDT 2012
Author: bebuild
Date: Wed Oct 3 09:15:24 2012
New Revision: 374294
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=374294
Log:
Merge changes for 1.8.17.0-rc2
This includes r373878 for ASTERISK-20390; r373165 and r373532 for ASTERISK-20409.
Updated version and ChangeLog; removed old release summaries.
Removed:
tags/1.8.17.0-rc2/asterisk-1.8.17.0-rc1-summary.html
tags/1.8.17.0-rc2/asterisk-1.8.17.0-rc1-summary.txt
Modified:
tags/1.8.17.0-rc2/ (props changed)
tags/1.8.17.0-rc2/.version
tags/1.8.17.0-rc2/ChangeLog
tags/1.8.17.0-rc2/channels/chan_local.c
tags/1.8.17.0-rc2/channels/chan_sip.c
Propchange: tags/1.8.17.0-rc2/
------------------------------------------------------------------------------
svn:mergeinfo = /branches/1.8:373165,373532,373878
Modified: tags/1.8.17.0-rc2/.version
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.17.0-rc2/.version?view=diff&rev=374294&r1=374293&r2=374294
==============================================================================
--- tags/1.8.17.0-rc2/.version (original)
+++ tags/1.8.17.0-rc2/.version Wed Oct 3 09:15:24 2012
@@ -1,1 +1,1 @@
-1.8.17.0-rc1
+1.8.17.0-rc2
Modified: tags/1.8.17.0-rc2/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.17.0-rc2/ChangeLog?view=diff&rev=374294&r1=374293&r2=374294
==============================================================================
--- tags/1.8.17.0-rc2/ChangeLog (original)
+++ tags/1.8.17.0-rc2/ChangeLog Wed Oct 3 09:15:24 2012
@@ -1,3 +1,31 @@
+2012-10-03 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.17.0-rc2 Released.
+
+ * Fix an issue where Local channels dialed by app_queue are considered
+ in use immediately.
+
+ The chan_local channel driver returns a device state of in use even
+ if a created Local channel has not yet been dialed. This fix
+ changes the logic to return a state of not in use until the channel
+ itself has been dialed.
+
+ (closes issue ASTERISK-20390)
+ Reported by: time_ringenbach
+
+ * Fix a regression where direct media was not permitted for calls
+ using SIP INFO DTMF
+
+ A change was committed to fix direct media ACL support. This change
+ wrongly assumed that only a single channel technology structure
+ exists for chan_sip. This is in fact false as a second exists for
+ calls using SIP INFO DTMF. The code which performs direct media ACL
+ checking now checks for both the non-INFO DTMF and INFO DTMF channel
+ technology structures.
+
+ (closes issue ASTERISK-20409)
+ Reported by: michele ciciotti privatewave
+
2012-09-13 Asterisk Development Team <asteriskteam at digium.com>
* Asterisk 1.8.17.0-rc1 Released.
Modified: tags/1.8.17.0-rc2/channels/chan_local.c
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.17.0-rc2/channels/chan_local.c?view=diff&rev=374294&r1=374293&r2=374294
==============================================================================
--- tags/1.8.17.0-rc2/channels/chan_local.c (original)
+++ tags/1.8.17.0-rc2/channels/chan_local.c Wed Oct 3 09:15:24 2012
@@ -308,11 +308,13 @@
res = AST_DEVICE_NOT_INUSE;
it = ao2_iterator_init(locals, 0);
- while ((lp = ao2_iterator_next(&it))) {
+ while ((lp = ao2_iterator_next(&it)) && (res == AST_DEVICE_NOT_INUSE)) {
if (!strcmp(exten, lp->exten) && !strcmp(context, lp->context) && lp->owner) {
- res = AST_DEVICE_INUSE;
- ao2_ref(lp, -1);
- break;
+ ao2_lock(lp);
+ if (ast_test_flag(lp, LOCAL_LAUNCHED_PBX)) {
+ res = AST_DEVICE_INUSE;
+ }
+ ao2_unlock(lp);
}
ao2_ref(lp, -1);
}
Modified: tags/1.8.17.0-rc2/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.17.0-rc2/channels/chan_sip.c?view=diff&rev=374294&r1=374293&r2=374294
==============================================================================
--- tags/1.8.17.0-rc2/channels/chan_sip.c (original)
+++ tags/1.8.17.0-rc2/channels/chan_sip.c Wed Oct 3 09:15:24 2012
@@ -29253,7 +29253,8 @@
if (!(opp_chan = ast_bridged_channel(chan))) {
return NULL;
- } else if ((opp_chan->tech != &sip_tech) || (!(opp = opp_chan->tech_pvt))) {
+ } else if (((opp_chan->tech != &sip_tech) && (opp_chan->tech != &sip_tech_info)) ||
+ (!(opp = opp_chan->tech_pvt))) {
return NULL;
}
@@ -29331,7 +29332,8 @@
if (!(opp_chan = ast_bridged_channel(chan))) {
return AST_RTP_GLUE_RESULT_FORBID;
- } else if ((opp_chan->tech != &sip_tech) || (!(opp = opp_chan->tech_pvt))) {
+ } else if (((opp_chan->tech != &sip_tech) && (opp_chan->tech != &sip_tech_info)) ||
+ (!(opp = opp_chan->tech_pvt))) {
return AST_RTP_GLUE_RESULT_FORBID;
}
@@ -29386,7 +29388,8 @@
if (!(opp_chan = ast_bridged_channel(chan))) {
return AST_RTP_GLUE_RESULT_FORBID;
- } else if ((opp_chan->tech != &sip_tech) || (!(opp = opp_chan->tech_pvt))) {
+ } else if (((opp_chan->tech != &sip_tech) && (opp_chan->tech != &sip_tech_info)) ||
+ (!(opp = opp_chan->tech_pvt))) {
return AST_RTP_GLUE_RESULT_FORBID;
}
@@ -29432,7 +29435,8 @@
if (!(opp_chan = ast_bridged_channel(chan))) {
return AST_RTP_GLUE_RESULT_FORBID;
- } else if ((opp_chan->tech != &sip_tech) || (!(opp = opp_chan->tech_pvt))) {
+ } else if (((opp_chan->tech != &sip_tech) && (opp_chan->tech != &sip_tech_info)) ||
+ (!(opp = opp_chan->tech_pvt))) {
return AST_RTP_GLUE_RESULT_FORBID;
}
More information about the asterisk-commits
mailing list