[asterisk-bugs] [Asterisk 0013467]: Seg fault 1.6.0 trunk

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13467 
====================================================================== 
Reported By:                edantie
Assigned To:                murf
====================================================================== 
Project:                    Asterisk
Issue ID:                   13467
Category:                   CDR/General
Reproducibility:            sometimes
Severity:                   crash
Priority:                   normal
Status:                     feedback
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 142733 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-09-12 04:38 CDT
Last Modified:              2008-09-23 11:32 CDT
====================================================================== 
Summary:                    Seg fault 1.6.0 trunk
Description: 
Program terminated with signal 11, Segmentation fault.
http://bugs.digium.com/view.php?id=0  0x0000000000446ed3 in ast_cdr_start
(cdr=0x4a0) at cdr.c:680
680			if (!ast_test_flag(cdr, AST_CDR_FLAG_LOCKED)) {

======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0013489 Asterisk crash 1.4 SVN
related to          0013490 crash in ast_cdr_start, backtraces atta...
====================================================================== 

---------------------------------------------------------------------- 
 (0092795) svnbot (reporter) - 2008-09-23 11:32
 http://bugs.digium.com/view.php?id=13467#c92795 
---------------------------------------------------------------------- 
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: 0092795                          
======================================================================