[asterisk-bugs] [Zaptel 0009379]: [patch] zt_specchan_open race because ZT_FLAG_OPEN testing and setting is not atomic
noreply at bugs.digium.com
noreply at bugs.digium.com
Thu Jun 5 12:52:25 CDT 2008
A NOTE has been added to this issue.
======================================================================
http://bugs.digium.com/view.php?id=9379
======================================================================
Reported By: Matti
Assigned To: mattf
======================================================================
Project: Zaptel
Issue ID: 9379
Category: zaptel (the module)
Reproducibility: have not tried
Severity: minor
Priority: normal
Status: assigned
Zaptel Version: 1.4.7.1
SVN Branch (only for SVN checkouts, not tarball releases): N/A
SVN Revision (number only!):
Disclaimer on File?: No
Request Review:
======================================================================
Date Submitted: 03-26-2007 07:55 CDT
Last Modified: 06-05-2008 12:52 CDT
======================================================================
Summary: [patch] zt_specchan_open race because ZT_FLAG_OPEN
testing and setting is not atomic
Description:
Function zt_specchan_open in file zaptel.c can have a race condition
because ZT_FLAG_OPEN testing and setting is not atomic. The function should
check and set the flag holding the spin lock continuously in and between
the following lines:
if (chans[unit]->flags & ZT_FLAG_OPEN)
chans[unit]->flags |= ZT_FLAG_OPEN;
======================================================================
Relationships ID Summary
----------------------------------------------------------------------
related to 0010806 Channel opening and unregistering race
======================================================================
----------------------------------------------------------------------
tzafrir - 06-05-08 12:52
----------------------------------------------------------------------
ok, just check that we don't look at an invalid channel after the close:
clear_check.diff
Issue History
Date Modified Username Field Change
======================================================================
06-05-08 12:52 tzafrir Note Added: 0087867
======================================================================
More information about the asterisk-bugs
mailing list