[asterisk-dev] Re: [svn-commits] file: branch 1.4
r319- /branches/1.4/asterisk-ooh323c/src/chan_h323.c
Dan Austin
Dan_Austin at Phoenix.com
Thu Dec 14 17:15:50 MST 2006
Russell wrote:
>> --- branches/1.4/asterisk-ooh323c/src/chan_h323.c (original)
>> +++ branches/1.4/asterisk-ooh323c/src/chan_h323.c Thu Dec 14 16:19:53
2006
>> @@ -247,15 +247,13 @@
>>
>> /* Don't hold a h323 pvt lock while we allocate a channel */
>> ast_mutex_unlock(&i->lock);
>> - ch = ast_channel_alloc(1);
>> + ch = ast_channel_alloc(1, state, i->callerid_num,
i->callerid_name, "OOH323/%s-%04x", host, rand() & 0xffff);
>> ast_mutex_lock(&i->lock);
>>
>> if(ch)
>> {
>> ast_mutex_lock(&ch->lock);
>> ch->tech = &ooh323_tech;
>> - ast_string_field_build(ch, name, "OOH323/%s-%04x",
>> - host, rand() &
0xffff);
>>
>> ch->nativeformats = i->capability;
> Just as you removed the code that manually set the channel name, the
code
> that sets the channel state and callerid information needs to be
removed
> as well.
My appologies. I spotted chan_ooh323 not building and had to dig
through
five weeks of updates to find the change to ast_channel_alloc and why.
I did not notice the changes to state and channel name, and file was
covering
my half-assed patch....
I can get another cmall patch up on mantis if that would be the best
thing
to do.
Dan
More information about the asterisk-dev
mailing list