[Home]

Summary:ASTERISK-11492: Mixmonitor crashing asterisk randomly
Reporter:Fabiano Heringer (fabianoheringer)Labels:
Date Opened:2008-02-21 13:42:40.000-0600Date Closed:2008-04-01 11:52:32
Priority:CriticalRegression?No
Status:Closed/CompleteComponents:Applications/app_mixmonitor
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 12047.diff
Description:Asterisk is crashed by mixmonitor in ramdonly cases, I´m using Zap Channels, Sip and E1 bank channels and getting this error

That´s a in issue related about this, but it´s using ChanSpy, I`m not using chanspy. In past week I reported this issue, but I was using the rc3 version, the administrator answered to me that release 1.4.18 version was corrected this,but same crash!

Using 1.4.18 release version

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

#0  0x080f1393 in ast_slinfactory_read (sf=0x8273ae4, buf=0xb37b50a0, samples=160) at slinfactory.c:128
128                     if ((frame_ptr = AST_LIST_REMOVE_HEAD(&sf->queue, frame_list))) {
(gdb) bt full
#0  0x080f1393 in ast_slinfactory_read (sf=0x8273ae4, buf=0xb37b50a0, samples=160) at slinfactory.c:128
       cur = (struct ast_frame *) 0xffe80010
       sofar = 0
       ineed = 160
       remain = 4294508540
       frame_data = (short int *) 0x82ae880
       offset = (short int *) 0xb37b50a0
#1  0x080715f8 in ast_audiohook_read_frame (audiohook=0x8273070, samples=160, direction=AST_AUDIOHOOK_DIRECTION_READ,
   format=64) at audiohook.c:213
       final_frame = (struct ast_frame *) 0x0
#2  0xb669a630 in mixmonitor_thread (obj=0x8273070) at app_mixmonitor.c:166
       fr = (struct ast_frame *) 0xa0
       fs = (struct ast_filestream *) 0x8283600
       oflags = 0
       ext = 0x0
       errflag = 0
       __PRETTY_FUNCTION__ = "mixmonitor_thread"
#3  0x080fda15 in dummy_start (data=0x8273ae4) at utils.c:852
       _buffer = {__routine = 0x80684b0 <ast_unregister_thread>, __arg = 0xb37b5bb0, __canceltype = 0, __prev = 0x0}
       ret = (void *) 0x829bfb8
       a = {start_routine = 0xb669a4a0 <mixmonitor_thread>, data = 0x8273070,
 name = 0x829bfb8 "mixmonitor_thread    started at [  286] app_mixmonitor.c launch_monitor_thread()"}
#4  0xb7ee120e in start_thread () from /lib/tls/libpthread.so.0
No symbol table info available.
ASTERISK-1  0xb7e3b0de in clone () from /lib/tls/libc.so.6
No symbol table info available.
Comments:By: jmls (jmls) 2008-02-26 04:57:24.000-0600

I think that MixMonitor uses the same code as chanspy, so perhaps the problem is the same.

Having said that, there has been a recent commit to 1.4 svn to fix chanspy, so perhaps this would also fix this issue.

Would you please download the latest 1.4 svn code, and test that.

Thanks.
By: Joshua C. Colp (jcolp) 2008-02-26 10:36:40.000-0600

Would it be possible to get SSH access so I can examine this core dump more closely?
By: Fabiano Heringer (fabianoheringer) 2008-02-26 10:47:36.000-0600

Yes, sure...please give an private mail or msn,icq,gtalk or another way to contact you, please mail me in fabiano@grupoheringer.com.br
By: Fabiano Heringer (fabianoheringer) 2008-03-24 07:05:15

Hi, Sorry in that day I could not got access to you in my server, please mail me again in fabianoheringer@hotmail.com to connect again in SSH, I reactivated the mixmonitor and now I have core dump file.
thanks
By: Joshua C. Colp (jcolp) 2008-03-24 10:05:22

Email has been sent requesting access.
By: Joshua C. Colp (jcolp) 2008-03-24 17:15:24

Please try the attached patch.
By: Fabiano Heringer (fabianoheringer) 2008-03-26 17:53:53

no crashes until now using the patch. I will test for more week and will report.
By: Jason Parker (jparker) 2008-03-31 18:23:39

It's been almost a week.  Has this patch still been working for you?
By: Fabiano Heringer (fabianoheringer) 2008-03-31 23:59:15

YES, no crash for last week...it´s look like solved problem
By: Digium Subversion (svnbot) 2008-04-01 11:40:45

Repository: asterisk
Revision: 112125

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

------------------------------------------------------------------------
r112125 | file | 2008-04-01 11:40:44 -0500 (Tue, 01 Apr 2008) | 5 lines

Ensure that we do not exceed the hold's maximum size with a single frame.
(closes issue ASTERISK-11492)
Reported by: fabianoheringer
Tested by: fabianoheringer

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

http://svn.digium.com/view/asterisk?view=rev&revision=112125
By: Digium Subversion (svnbot) 2008-04-01 11:46:07

Repository: asterisk
Revision: 112126

_U  trunk/
U   trunk/include/asterisk/slinfactory.h
U   trunk/main/slinfactory.c

------------------------------------------------------------------------
r112126 | file | 2008-04-01 11:46:06 -0500 (Tue, 01 Apr 2008) | 13 lines

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

........
r112125 | file | 2008-04-01 13:45:14 -0300 (Tue, 01 Apr 2008) | 5 lines

Ensure that we do not exceed the hold's maximum size with a single frame.
(closes issue ASTERISK-11492)
Reported by: fabianoheringer
Tested by: fabianoheringer

........

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

http://svn.digium.com/view/asterisk?view=rev&revision=112126
By: Digium Subversion (svnbot) 2008-04-01 11:52:32

Repository: asterisk
Revision: 112127

_U  branches/1.6.0/
U   branches/1.6.0/include/asterisk/slinfactory.h
U   branches/1.6.0/main/slinfactory.c

------------------------------------------------------------------------
r112127 | file | 2008-04-01 11:52:32 -0500 (Tue, 01 Apr 2008) | 21 lines

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

................
r112126 | file | 2008-04-01 13:50:37 -0300 (Tue, 01 Apr 2008) | 13 lines

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

........
r112125 | file | 2008-04-01 13:45:14 -0300 (Tue, 01 Apr 2008) | 5 lines

Ensure that we do not exceed the hold's maximum size with a single frame.
(closes issue ASTERISK-11492)
Reported by: fabianoheringer
Tested by: fabianoheringer

........

................

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

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