diff options
| author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-25 17:19:39 +0000 |
|---|---|---|
| committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-25 17:19:39 +0000 |
| commit | 1c852c16ad8f394f8e6b8793286ca199630a1a88 (patch) | |
| tree | 2cd3fa538b72b1c5dff3ed9c8a14dbc083d6637b /main/channel.c | |
| parent | 958638e4ee206b2b2b18d79d939518bc3c0336a2 (diff) | |
Fix some errant device states by making the devicestate API more strict in
terms of the device argument (only without the unique identifier appended).
(closes issue #12771)
Reported by: davidw
Patches:
20080717__bug12771.diff.txt uploaded by Corydon76 (license 14)
Tested by: davidw, jvandal, murf
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@133649 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main/channel.c')
| -rw-r--r-- | main/channel.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/main/channel.c b/main/channel.c index 9a5d70125..1383feb0f 100644 --- a/main/channel.c +++ b/main/channel.c @@ -1201,7 +1201,7 @@ void ast_channel_free(struct ast_channel *chan) struct ast_frame *f; struct varshead *headp; struct ast_datastore *datastore = NULL; - char name[AST_CHANNEL_NAME]; + char name[AST_CHANNEL_NAME], *dashptr; headp=&chan->varshead; @@ -1234,6 +1234,9 @@ void ast_channel_free(struct ast_channel *chan) sched_context_destroy(chan->sched); ast_copy_string(name, chan->name, sizeof(name)); + if ((dashptr = strrchr(name, '-'))) { + *dashptr = '\0'; + } /* Stop monitoring */ if (chan->monitor) @@ -3669,13 +3672,19 @@ void ast_set_callerid(struct ast_channel *chan, const char *callerid, const char int ast_setstate(struct ast_channel *chan, enum ast_channel_state state) { + char name[AST_CHANNEL_NAME], *dashptr; int oldstate = chan->_state; if (oldstate == state) return 0; + ast_copy_string(name, chan->name, sizeof(name)); + if ((dashptr = strrchr(name, '-'))) { + *dashptr = '\0'; + } + chan->_state = state; - ast_device_state_changed_literal(chan->name); + ast_device_state_changed_literal(name); /* setstate used to conditionally report Newchannel; this is no more */ manager_event(EVENT_FLAG_CALL, "Newstate", |