diff options
| author | Matthew Jordan <mjordan@digium.com> | 2013-10-03 21:46:07 +0000 |
|---|---|---|
| committer | Matthew Jordan <mjordan@digium.com> | 2013-10-03 21:46:07 +0000 |
| commit | facebb3f3cd4065436bf0060417fc126e7678ba6 (patch) | |
| tree | 35c82823780fa3214b650932ad7361fcce36691d | |
| parent | b11983d4809327c483bfce23efca53bc0fb42c89 (diff) | |
Remove publication of a channel snapshot when the technology is set
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
| -rw-r--r-- | main/channel_internal_api.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/main/channel_internal_api.c b/main/channel_internal_api.c index de2cc9c71..51d83e12c 100644 --- a/main/channel_internal_api.c +++ b/main/channel_internal_api.c @@ -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) { |