[asterisk-bugs] [Asterisk 0017052]: chanspy on channel in MeetMe conference results in a crash

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17052 
====================================================================== 
Reported By:                dvossel
Assigned To:                dvossel
====================================================================== 
Project:                    Asterisk
Issue ID:                   17052
Category:                   General
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                       SWP-1106 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-03-17 12:54 CDT
Last Modified:              2010-04-29 10:52 CDT
====================================================================== 
Summary:                    chanspy on channel in MeetMe conference results in a
crash
Description: 
Using ChanSpy on channel in that is in a MeetMe conference results in a
crash.  This occurs in both sip and iax so I don't believe it has to do
with a channel driver.  bt and bt full included.

I put this in the general category because I really have no idea what is
causing the crash.  I don't thing it has to do with chanspy, because
chanspy works correctly when spying on channels outside of the MeetMe room.
 Perhaps it has something to do with the channel in the meetme room being
bridged to a local channel.  I'm not sure.

I can reproduce this 100% of the time.  Just put two channels in a meetme
conference (using ulaw as codec not sure if that matters) and then
attempting to use ChanSpy on either of those channels will result in a
crash.
====================================================================== 

---------------------------------------------------------------------- 
 (0121162) svnbot (reporter) - 2010-04-29 10:52
 https://issues.asterisk.org/view.php?id=17052#c121162 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 260049

U   branches/1.4/include/asterisk/audiohook.h
U   branches/1.4/main/audiohook.c

------------------------------------------------------------------------
r260049 | dvossel | 2010-04-29 10:31:02 -0500 (Thu, 29 Apr 2010) | 17
lines

Fixes crash in audiohook_write_list

The middle_frame in the audiohook_write_list function was
being freed if a audiohook manipulator returned a failure.
This is incorrect logic.  This patch resolves this and
adds detailed descriptions of how this function should work
and why manipulator failures must be ignored.

(closes issue https://issues.asterisk.org/view.php?id=17052)
Reported by: dvossel
Tested by: dvossel

(closes issue https://issues.asterisk.org/view.php?id=16196)
Reported by: atis

Review: https://reviewboard.asterisk.org/r/623/

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=260049 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-04-29 10:52 svnbot         Checkin                                      
2010-04-29 10:52 svnbot         Note Added: 0121162                          
======================================================================