[Home]

Summary:ASTERISK-14646: segmentation fault when using mixmonitor with two calls
Reporter:John Covici (covici)Labels:
Date Opened:2009-08-13 05:46:39Date Closed:2009-11-10 15:57:31.000-0600
Priority:CriticalRegression?No
Status:Closed/CompleteComponents:Core/Jitterbuffer
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) backtrace.txt
( 1) backtrace2.txt
Description:If I use the *1 combination to start mixmonitor on my analog phone connected via an ata,  and another call is going on using my soft phone, I get a segmentation fault, after it complains about writing a non voice frame to the stream.

****** ADDITIONAL INFORMATION ******

Here are the log entries:

[Aug 12 18:01:59] DTMF[14702] channel.c: DTMF end emulation of '1' queued on SIP/202-b792b8d0
[Aug 12 18:01:59] VERBOSE[14702] logger.c:     -- <SIP/202-b792b8d0> Playing 'beep' (language 'en')
[Aug 12 18:02:00] VERBOSE[14702] logger.c:     -- User hit '*1' to record call. filename: wav|auto-1250114520-7037775986-*2519162330500|m
[Aug 12 18:02:00] WARNING[14702] file.c: Tried to write non-voice frame
[Aug 12 18:02:00] WARNING[14702] channel.c: Failed to write data to channel monitor write stream
[Aug 12 18:02:00] WARNING[14702] file.c: Tried to write non-voice frame
[Aug 12 18:02:00] WARNING[14702] channel.c: Failed to write data to channel monitor write stream
Comments:By: David Brillert (aragon) 2009-08-13 11:15:42

This looks a lot like bug https://issues.asterisk.org/view.php?id=15560
Try disabling your jitterbuffer and see if it still crashes.
By: John Covici (covici) 2009-08-13 11:32:29

jitter buffering is not enabled -- at least according to  the configs jitter buffer is not mentioned and it seems the default is disabled.
By: Leif Madsen (lmadsen) 2009-08-20 16:07:57

Can you provide the dialplan that would allow us to reproduce this issue? Thanks!
By: John Covici (covici) 2009-08-20 19:51:45

This may be difficult, as this is a freepbx installation, so I am not sure how much of the dialplan I should give you, the whole thing would be quite a number of files -- perhaps the previous 10 minutes of the logs might help?
By: John Covici (covici) 2009-08-24 15:14:35

By trial and error binary search, I found that 209991 is the commit which causes this bug.  I was thinking of diffing out the mix monitor portion, but this is a large one and this may have unintended consequences.  I hope this helps to resolve this one.
By: John Covici (covici) 2009-09-04 07:32:27

typographical error in my last note, its 200991.
By: John Covici (covici) 2009-09-06 06:22:44

If I update to 216435 which is the latest I can get, then one-touch record fails always -- at least from my sip/at phone.  Looks like garbage in the frame list.  Here is a backtrace log.  I don't need two calls, just hitting *1 on my analog phone connected by an ata will cause the problem.


By: John Covici (covici) 2009-09-08 17:20:28

why is the status still feedback?
By: John Covici (covici) 2009-09-08 17:20:59

I tried using 1.6.0 and still have the problem.
By: Leif Madsen (lmadsen) 2009-09-08 18:29:49

It was still in feedback status because the status is update manually by a bug marshal, and one had not seen that you updated the issue yet (i.e. me).
By: Leif Madsen (lmadsen) 2009-09-08 18:30:44

Assigned to Tilghman to review the backtrace. Let me know if you need anything from me in terms of testing. Thanks!
By: John Covici (covici) 2009-09-09 07:29:46

I have discovered that some sip setting is involved in this bug -- I updated freepbx and got rid of all the sip general settings and redid hem all with their new module and lo and behold it now works.  This is 1.6.0 svn 217034.  Not sure which setting it was, but I did notice that jitter buffers are no longer being created.
By: Leif Madsen (lmadsen) 2009-09-09 07:32:30

Tilghman, please close this issue unless it is clear from the backtrace what the issue was.

If the reporter can determine the difference between the two sip.conf files they were running, that would be great, but without the information about the specific change in the sip.conf that was generated, there isn't going to be much we can do about this issue.
By: John Covici (covici) 2009-09-09 11:28:58

OK, the difference seems to be if I have jitter buffers enabled and forced, then we get the seg fault, if I take them away the seg fault goes away.
By: David Brillert (aragon) 2009-09-09 11:43:17

This is why I mentioned this in my previous note.
Maybe a bug marshall could make a relation of this ticket to ASTERISK-14517

*This looks a lot like bug https://issues.asterisk.org/view.php?id=15560 [^]
Try disabling your jitterbuffer and see if it still crashes.*
By: Leif Madsen (lmadsen) 2009-09-09 11:56:58

OK, updated and marked as related to 15560 (although it is most likely a duplicate)
By: Digium Subversion (svnbot) 2009-11-06 10:46:58.000-0600

Repository: asterisk
Revision: 228409

U   branches/1.4/main/abstract_jb.c

------------------------------------------------------------------------
r228409 | file | 2009-11-06 10:46:56 -0600 (Fri, 06 Nov 2009) | 7 lines

Fix a bug caused by a partially invalid frame (from the jitterbuffer) passing through the Asterisk core.

(closes issue ASTERISK-14517)
Reported by: jvandal
(closes issue ASTERISK-14646)
Reported by: covici

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

http://svn.digium.com/view/asterisk?view=rev&revision=228409
By: Digium Subversion (svnbot) 2009-11-06 10:48:00.000-0600

Repository: asterisk
Revision: 228410

_U  trunk/
U   trunk/main/abstract_jb.c

------------------------------------------------------------------------
r228410 | file | 2009-11-06 10:47:58 -0600 (Fri, 06 Nov 2009) | 14 lines

Merged revisions 228409 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
 r228409 | file | 2009-11-06 12:41:20 -0400 (Fri, 06 Nov 2009) | 7 lines
 
 Fix a bug caused by a partially invalid frame (from the jitterbuffer) passing through the Asterisk core.
 
 (closes issue ASTERISK-14517)
 Reported by: jvandal
 (closes issue ASTERISK-14646)
 Reported by: covici
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=228410
By: Digium Subversion (svnbot) 2009-11-06 10:49:13.000-0600

Repository: asterisk
Revision: 228411

_U  branches/1.6.0/
U   branches/1.6.0/main/abstract_jb.c

------------------------------------------------------------------------
r228411 | file | 2009-11-06 10:49:12 -0600 (Fri, 06 Nov 2009) | 21 lines

Merged revisions 228410 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r228410 | file | 2009-11-06 12:42:23 -0400 (Fri, 06 Nov 2009) | 14 lines
 
 Merged revisions 228409 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r228409 | file | 2009-11-06 12:41:20 -0400 (Fri, 06 Nov 2009) | 7 lines
   
   Fix a bug caused by a partially invalid frame (from the jitterbuffer) passing through the Asterisk core.
   
   (closes issue ASTERISK-14517)
   Reported by: jvandal
   (closes issue ASTERISK-14646)
   Reported by: covici
 ........
................

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

http://svn.digium.com/view/asterisk?view=rev&revision=228411
By: Digium Subversion (svnbot) 2009-11-06 10:50:00.000-0600

Repository: asterisk
Revision: 228412

_U  branches/1.6.1/
U   branches/1.6.1/main/abstract_jb.c

------------------------------------------------------------------------
r228412 | file | 2009-11-06 10:49:58 -0600 (Fri, 06 Nov 2009) | 21 lines

Merged revisions 228410 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r228410 | file | 2009-11-06 12:42:23 -0400 (Fri, 06 Nov 2009) | 14 lines
 
 Merged revisions 228409 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r228409 | file | 2009-11-06 12:41:20 -0400 (Fri, 06 Nov 2009) | 7 lines
   
   Fix a bug caused by a partially invalid frame (from the jitterbuffer) passing through the Asterisk core.
   
   (closes issue ASTERISK-14517)
   Reported by: jvandal
   (closes issue ASTERISK-14646)
   Reported by: covici
 ........
................

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

http://svn.digium.com/view/asterisk?view=rev&revision=228412
By: Digium Subversion (svnbot) 2009-11-06 10:50:34.000-0600

Repository: asterisk
Revision: 228413

_U  branches/1.6.2/
U   branches/1.6.2/main/abstract_jb.c

------------------------------------------------------------------------
r228413 | file | 2009-11-06 10:50:32 -0600 (Fri, 06 Nov 2009) | 21 lines

Merged revisions 228410 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r228410 | file | 2009-11-06 12:42:23 -0400 (Fri, 06 Nov 2009) | 14 lines
 
 Merged revisions 228409 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r228409 | file | 2009-11-06 12:41:20 -0400 (Fri, 06 Nov 2009) | 7 lines
   
   Fix a bug caused by a partially invalid frame (from the jitterbuffer) passing through the Asterisk core.
   
   (closes issue ASTERISK-14517)
   Reported by: jvandal
   (closes issue ASTERISK-14646)
   Reported by: covici
 ........
................

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

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