diff options
| author | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-08-05 05:08:50 +0000 |
|---|---|---|
| committer | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-08-05 05:08:50 +0000 |
| commit | f05aa97fe5ee633cffc5641a9395f0c06ca27de5 (patch) | |
| tree | ae71fb6b76d48d499d0f8c8db72cfb68cc837b7f /channels/chan_modem.c | |
| parent | 85d8708df8c31feb766348a0772cc0c652765aa5 (diff) | |
Fix an issue that would cause a NewCallerID manager event to be generated
before the channel's NewChannel event. This was due to a somewhat recent
change that included using ast_set_callerid() where it wasn't before. This
function should not be used in the channel driver "new" functions.
(issue #7654, fixed by me)
Also, fix a couple minor bugs in usecount handling. chan_iax2 could have
increased the usecount but then returned an error. The place where chan_sip
increased the usecount did not call ast_update_usecount()
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.2@38904 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels/chan_modem.c')
| -rw-r--r-- | channels/chan_modem.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/channels/chan_modem.c b/channels/chan_modem.c index 6718db141..8be355ea5 100644 --- a/channels/chan_modem.c +++ b/channels/chan_modem.c @@ -573,7 +573,12 @@ struct ast_channel *ast_modem_new(struct ast_modem_pvt *i, int state) tmp->tech_pvt = i; strncpy(tmp->context, i->context, sizeof(tmp->context)-1); - ast_set_callerid(tmp, i->cid_num, i->cid_name, i->cid_num); + if (!ast_strlen_zero(i->cid_num)) { + tmp->cid.cid_num = strdup(i->cid_num); + tmp->cid.cid_ani = strdup(i->cid_num); + } + if (!ast_strlen_zero(i->cid_name)) + tmp->cid.cid_name = strdup(i->cid_name); if (!ast_strlen_zero(i->language)) strncpy(tmp->language,i->language, sizeof(tmp->language)-1); |