[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

[Joshua Colp]

> On May 1, 2014, 6: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?

Or to be more pedantic: by the code that was invoked when the function was entered.


- Joshua


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


On April 29, 2014, 10: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, 10: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/20140501/b7164a90/attachment.html>