[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-12-2008 05:51 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.
====================================================================== 

---------------------------------------------------------------------- 
 festr - 07-12-08 05:51  
---------------------------------------------------------------------- 
Look at main/audiogook.c "$Revision: 130236 $" at function
ast_audiohook_write_frame (line 132)

there is assign 

        struct timeval *time = (direction == AST_AUDIOHOOK_DIRECTION_READ
? &audiohook->read_time : &audiohook->write_time), previous_time = *time;

followed immediately by "*time = ast_tvnow();"

this mean, that previous_time could be different than the actual time. Is
this ok? 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-12-08 05:51  festr          Note Added: 0090140                          
======================================================================