[asterisk-bugs] [Asterisk 0012837]: Chanspy audio is delayed or lost

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12837 
====================================================================== 
Reported By:                pmathews
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12837
Category:                   Applications/app_chanspy
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.4.20.1 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             06-11-2008 16:49 CDT
Last Modified:              07-14-2008 16:24 CDT
====================================================================== 
Summary:                    Chanspy audio is delayed or lost
Description: 
If extension (A) is being monitored via chanspy on extension (C) and (A)
places a call (B) on hold, the audio from the other side of the call (B) is
either delayed or lost to station (C) when the extension being monitored
(A) returns from placing a call on hold. This does not happen if extension
(C) is monitoring extension (B) and extension (A) places extension (B) on
hold.
====================================================================== 

---------------------------------------------------------------------- 
 putnopvut - 07-14-08 16:24  
---------------------------------------------------------------------- 
I've been looking into audiohook code based on issue 13005. I can address
some of the concerns raised here.

1. I'm not sure if the AST_AUDIOHOOK_TRIGGER_SYNC flag is necessary for
chanspy. 

2. The fact that the "time" and "previous_time" variables are different is
intentional. If you look below, you'll see that there is a call to
ast_tvdiff that uses those two values.

What makes this and other audiohook-related problems difficult to fix is
that I never can seem to reproduce the problem locally, so I can't be sure
if the patch I'm writing will necessarily fix the issue.

Here's what I suggest. Look at issue http://bugs.digium.com/view.php?id=13005.
Patch your source with
13005.patch. That was a fix for a typo that was in audiohook.c. If that
helps, then let me know. If it doesn't, then I will work on making a
trigger-related audiohook patch for chanspy. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-14-08 16:24  putnopvut      Note Added: 0090242                          
======================================================================