]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Remove publication of a channel snapshot when the technology is set
authorMatthew Jordan <mjordan@digium.com>
Thu, 3 Oct 2013 21:40:20 +0000 (21:40 +0000)
committerMatthew Jordan <mjordan@digium.com>
Thu, 3 Oct 2013 21:40:20 +0000 (21:40 +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.

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

main/channel_internal_api.c

index de2cc9c71d042dcdfb51e1d6c5c6e346bd2560e1..51d83e12cd553e7ce6a03f7f5d42c921ec73974b 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)
 {