Remove publication of a channel snapshot when the technology is set
authorMatthew Jordan <mjordan@digium.com>
Thu, 3 Oct 2013 21:46:07 +0000 (21:46 +0000)
committerMatthew Jordan <mjordan@digium.com>
Thu, 3 Oct 2013 21:46:07 +0000 (21:46 +0000)
This patch removes said publication for a few reasons:
(1) It is unnecessary. Association of the channel technology with a specific
channel is an implementation detail that should be assumed to "just happen",
and consumers of Stasis don't need to be informed about it.
(2) Publication of said message can now cause crashes, as the actual creation
of a channel in normal locations now stages its messages. As a result, things
that create dummy channels (such as the SIP RTP QOS unit test) and associate
them with a channel technology were now crashing, as the channel itself was
not known by Stasis.
........

Merged revisions 400460 from http://svn.asterisk.org/svn/asterisk/branches/12

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@400461 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/channel_internal_api.c

index de2cc9c..51d83e1 100644 (file)
@@ -879,7 +879,6 @@ const struct ast_channel_tech *ast_channel_tech(const struct ast_channel *chan)
 void ast_channel_tech_set(struct ast_channel *chan, const struct ast_channel_tech *value)
 {
        chan->tech = value;
-       ast_channel_publish_snapshot(chan);
 }
 enum ast_channel_adsicpe ast_channel_adsicpe(const struct ast_channel *chan)
 {