[asterisk-bugs] [Asterisk 0011780]: Asterisk crashes due to non-atomic check on chan_iax.c:schedule_delivery

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11780 
====================================================================== 
Reported By:                guillecabeza
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11780
Category:                   Channels/chan_iax2
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     new
Asterisk Version:           1.4.17 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 93668 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             01-16-2008 15:54 CST
Last Modified:              02-13-2008 16:49 CST
====================================================================== 
Summary:                    Asterisk crashes due to non-atomic check on
chan_iax.c:schedule_delivery
Description: 
It may happen on hangup or if the user press a digit during the call(bridge
is temporarily on NULL) that this condition may be true for the first check
and false on the second one.

ast_bridged_channel(iaxs[fr->callno]->owner) &&
	    (ast_bridged_channel(iaxs[fr->callno]->owner)->tech->properties

Bringing asterisk down.
====================================================================== 

---------------------------------------------------------------------- 
 guillecabeza - 02-13-08 16:49  
---------------------------------------------------------------------- 
There is one parenthesis missing in the condition on the patch files I
uploaded, it should read

if( (!ast_test_flag(iaxs[fr->callno], IAX_FORCEJITTERBUF)) && owner &&
bridge && (bridge->tech->properties & AST_CHAN_TP_WANTSJITTER)) {

I may update the patches if and when someone takes a look at the problem. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
02-13-08 16:49  guillecabeza   Note Added: 0082203                          
======================================================================