[asterisk-dev] [Code Review] 3505: app_chanspy: Fix a bug where barge mode only works on the first connection when multiple sessions are spied on for a channel

Jonathan Rose reviewboard at asterisk.org
Fri May 2 15:33:27 CDT 2014



> On May 1, 2014, 1:14 p.m., Joshua Colp wrote:
> > /branches/11/apps/app_chanspy.c, lines 664-668
> > <https://reviewboard.asterisk.org/r/3505/diff/1/?file=58139#file58139line664>
> >
> >     Can you explain why the audiohook would not already be attached by the previous code here?
> 
> Joshua Colp wrote:
>     Or to be more pedantic: by the code that was invoked when the function was entered.

After some experimenting, it turns out there is no reason for the audiohook not to be attached by the previous call.  At least not after the changes that have been made. I'll be posting a new version shortly.


- Jonathan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3505/#review11799
-----------------------------------------------------------


On April 29, 2014, 5:07 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3505/
> -----------------------------------------------------------
> 
> (Updated April 29, 2014, 5:07 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23381
>     https://issues.asterisk.org/jira/browse/ASTERISK-23381
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This is a cleanup of the patches posted by Robert Moss for ASTERISK-23381 - A stalled review of the patch exists here: https://reviewboard.asterisk.org/r/3331/diff/1/#index_header
> 
> This version is made to apply to Asterisk 11 since the bug was first reported there.
> 
> 
> Diffs
> -----
> 
>   /branches/11/apps/app_chanspy.c 413071 
> 
> Diff: https://reviewboard.asterisk.org/r/3505/diff/
> 
> 
> Testing
> -------
> 
> 1. SIP/A calls a spy application which spies on SIP/B with the DTMF option (4 for spy mode, 5 for whisper mode, 6 for barge)
> 2. SIP/B calls SIP/C
> 3. Check to see if audio is flowing from A to B and from A to C... should start as neither. (A always hears both of the others though)
>  * press DTMF 5
>   - now audio should flow from A to B, but from not A to C
>  * press DTMF 6
>   - now audio should flow from A to B and A to C
> 4. Hang up SIP/B
> 5. Have SIP/B call SIP/C again
> 6. The spy should reconnect automatically and be in barge mode
>   - audio should flow from A to B and A to C
>  * press DTMF 5
>   - audio should flow from A to B, but not from A to C
>  * press DTMF 4
>   - audio should not flow from A to either phone
> 
> 
> Prior to patch: Audio from A to C could not be heard in the first check
> After the patch: Audio from A to C can be heard in the first check
> 
> All other checks work as expected in both situations.
> 
> 
> Thanks,
> 
> Jonathan Rose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140502/7d26b08a/attachment.html>


More information about the asterisk-dev mailing list