[asterisk-dev] [Code Review] Make iax2-provision use LIST macros and fix issue #13232

Mark Michelson mmichelson at digium.com
Wed Dec 3 13:10:24 CST 2008


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/72/#review174
-----------------------------------------------------------

Ship it!


Aside from my one comment below, this looks good to go!


/trunk/channels/iax2-provision.c
<http://reviewboard.digium.com/r/72/#comment335>

    Use ast_copy_string when copying strings.


- Mark


On 2008-12-03 11:21:02, Eliel Sardañons wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/72/
> -----------------------------------------------------------
> 
> (Updated 2008-12-03 11:21:02)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> I am trying to get this moving. I think uploading a review here will make more easy for others to review the patch.
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/iax2-provision.c 160446 
> 
> Diff: http://reviewboard.digium.com/r/72/diff
> 
> 
> Testing
> -------
> 
> Developer testing.
> 
> 
> Thanks,
> 
> Eliel
> 
>




More information about the asterisk-dev mailing list