diff options
author | dvossel <dvossel@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-01-08 19:43:50 +0000 |
---|---|---|
committer | dvossel <dvossel@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-01-08 19:43:50 +0000 |
commit | 34a43170d5d8de70edd6f5ca0fe9a0e6b65c8c92 (patch) | |
tree | 7a626639cb3a647429550b4b472ac9e45feb270e /include/asterisk/event.h | |
parent | f15d895f38ce6ac697aa96f1899173393cddbc64 (diff) |
Merged revisions 238635 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
........
r238635 | dvossel | 2010-01-08 13:39:30 -0600 (Fri, 08 Jan 2010) | 22 lines
fixes AUDIOHOOK_INHERIT regression
During the process of removing an audiohook from one channel
and attaching it to another the audiohook's status is updated
to DONE and then back to whatever it was previously. Typically
updating the status after setting it to DONE is not a good idea
because DONE can trigger unrecoverable audiohook destruction
events... because of this a conditional check was added to
audiohook_update_status to explicitly prevent the audiohook
from ever changing after being set to DONE. It was this check
that prevented audiohook inherit from work properly though.
Now ast_audiohook_move_by_source is treated as a special exception,
as the audiohook must be returned to its previous status after
attaching it to the new channel. This is only a safe operation
because the audiohook's lock is held the entire time, otherwise
this could cause trouble.
(closes issue #16522)
Reported by: corruptor
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@238637 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'include/asterisk/event.h')
0 files changed, 0 insertions, 0 deletions