[asterisk-dev] Re: [svn-commits] file: branch 1.4 r319 - /branches/1.4/asterisk-ooh323c/src/chan_h323.c

[Russell Bryant]

svn-commits at lists.digium.com wrote:
> Author: file
> Date: Thu Dec 14 16:19:53 2006
> New Revision: 319
> 
> URL: http://svn.digium.com/view/asterisk-addons?view=rev&rev=319
> Log:
> Update ast_channel_alloc call to new API format. (issue #8582 reported by DEA with minor mods by moi)
> 
> Modified:
>     branches/1.4/asterisk-ooh323c/src/chan_h323.c
> 
> Modified: branches/1.4/asterisk-ooh323c/src/chan_h323.c
> URL: http://svn.digium.com/view/asterisk-addons/branches/1.4/asterisk-ooh323c/src/chan_h323.c?view=diff&rev=319&r1=318&r2=319
> ==============================================================================
> --- 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.

-- 
Russell Bryant
Software Engineer
Digium, Inc.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: russell.vcf
Type: text/x-vcard
Size: 278 bytes
Desc: not available
Url : http://lists.digium.com/pipermail/asterisk-dev/attachments/20061214/f55fa1f8/russell.vcf