[Pkg-voip-commits] r8238 - in /asterisk/trunk/debian: changelog patches/dahdi_fxs_false_ringing patches/series

[tzafrir-guest at alioth.debian.org]

Author: tzafrir-guest
Date: Mon Mar 29 10:09:43 2010
New Revision: 8238

URL: http://svn.debian.org/wsvn/pkg-voip/?sc=1&rev=8238
Log:
Patch dahdi_fxs_false_ringing: Fix having Astribank FXS-s keep ringing if
answered too soon. 

Added:
    asterisk/trunk/debian/patches/dahdi_fxs_false_ringing
Modified:
    asterisk/trunk/debian/changelog
    asterisk/trunk/debian/patches/series

Modified: asterisk/trunk/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-voip/asterisk/trunk/debian/changelog?rev=8238&op=diff
==============================================================================
--- asterisk/trunk/debian/changelog (original)
+++ asterisk/trunk/debian/changelog Mon Mar 29 10:09:43 2010
@@ -11,8 +11,10 @@
   * Removed empty es/ and fr/ directories under sounds/ 
   * Patch settings_show_dirs: display the user values of more configurable
     items. 
-
- -- Tzafrir Cohen <tzafrir.cohen at xorcom.com>  Mon, 29 Mar 2010 12:53:18 +0300
+  * Patch dahdi_fxs_false_ringing: Fix having Astribank FXS-s keep ringing if
+    answered too soon. 
+
+ -- Tzafrir Cohen <tzafrir.cohen at xorcom.com>  Mon, 29 Mar 2010 12:56:08 +0300
 
 asterisk (1:1.6.2.2-1) unstable; urgency=medium
 

Added: asterisk/trunk/debian/patches/dahdi_fxs_false_ringing
URL: http://svn.debian.org/wsvn/pkg-voip/asterisk/trunk/debian/patches/dahdi_fxs_false_ringing?rev=8238&op=file
==============================================================================
--- asterisk/trunk/debian/patches/dahdi_fxs_false_ringing (added)
+++ asterisk/trunk/debian/patches/dahdi_fxs_false_ringing Mon Mar 29 10:09:43 2010
@@ -1,0 +1,22 @@
+Subject: Avoid incorrect re-switching to Ringing after call answered
+Bug: https://issues.asterisk.org/view.php?id=17067
+Author: Tzafrir Cohen <tzafrir.cohen at xorcom.com>
+
+As described in that bug report, Asterisk seems to insist on trying to set
+the state to ringing. It signals that the state needs setting 3 times. And
+unfortunately under certain conditions the last one of those gets delayed
+in the PCM stream and sets the state to Ringing only after the state was
+already set to Up.
+
+diff --git a/channels/chan_dahdi.c b/channels/chan_dahdi.c
+index f7338ae..a256f23 100644
+--- a/channels/chan_dahdi.c
++++ b/channels/chan_dahdi.c
+@@ -6068,6 +6072,7 @@ static struct ast_frame *dahdi_handle_event(struct ast_channel *ast)
+ 					p->subs[idx].f.frametype = AST_FRAME_CONTROL;
+ 					p->subs[idx].f.subclass = AST_CONTROL_ANSWER;
+ 					/* Make sure it stops ringing */
++					p->subs[SUB_REAL].needringing = 0;
+ 					dahdi_set_hook(p->subs[idx].dfd, DAHDI_OFFHOOK);
+ 					ast_debug(1, "channel %d answered\n", p->channel);
+ 					if (p->cidspill) {

Modified: asterisk/trunk/debian/patches/series
URL: http://svn.debian.org/wsvn/pkg-voip/asterisk/trunk/debian/patches/series?rev=8238&op=diff
==============================================================================
--- asterisk/trunk/debian/patches/series (original)
+++ asterisk/trunk/debian/patches/series Mon Mar 29 10:09:43 2010
@@ -18,3 +18,4 @@
 sound_files
 moh_datadir
 settings_show_dirs
+dahdi_fxs_false_ringing