[asterisk-bugs] [Asterisk 0013490]: crash in ast_cdr_start, backtraces attached

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13490 
====================================================================== 
Reported By:                seanbright
Assigned To:                murf
====================================================================== 
Project:                    Asterisk
Issue ID:                   13490
Category:                   CDR/General
Reproducibility:            random
Severity:                   crash
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-09-16 09:41 CDT
Last Modified:              2008-09-23 11:32 CDT
====================================================================== 
Summary:                    crash in ast_cdr_start, backtraces attached
Description: 
Getting a crash in our production PBX.  The codebase is based on a branch
of 1.4 that has some very minor modifications for functionality not related
to CDRs (ChanSpy mostly).  I can provide a patch against 1.4 if it is
really necessary, but I don't think it will be.  bt, bt full, and thread
apply bt full logs are coming shortly.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0013489 Asterisk crash 1.4 SVN
related to          0013467 Seg fault 1.6.0 trunk
related to          0013526 Last 1.4 svn crash. Backtrace attached
====================================================================== 

---------------------------------------------------------------------- 
 (0092794) svnbot (reporter) - 2008-09-23 11:32
 http://bugs.digium.com/view.php?id=13490#c92794 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 144066

U   branches/1.4/res/res_features.c

------------------------------------------------------------------------
r144066 | murf | 2008-09-23 11:32:04 -0500 (Tue, 23 Sep 2008) | 29 lines

(closes issue http://bugs.digium.com/view.php?id=13489)
Reported by: DougUDI
Tested by: murf

(closes issue http://bugs.digium.com/view.php?id=13490)
Reported by: seanbright
Tested by: murf

(closes issue http://bugs.digium.com/view.php?id=13467)
Reported by: edantie
Tested by: murf, edantie, DougUDI


This crash happens because we are unsafely handling old pointers.
The channel whose cdr is being handled, has been hung up and 
destroyed already. I reorganized the code a bit, and tried not
to lose the fork-cdr-chain concepts of the previous code.
I now verify that the 'previous' channel (the channel we
had when the bridge was started), still exists, by looking it up
by name in the channel list. I also do not try to reset the
CDR's of channels involved in bridges. 

Testing shows it solves the crash problem, and should not
negatively impact previous fixes involving CDR's generated
during/after blind transfers. (The reason we need to reset
the CDR's on the "beginning" channels in the first place).



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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-09-23 11:32 svnbot         Checkin                                      
2008-09-23 11:32 svnbot         Note Added: 0092794                          
======================================================================