[asterisk-dev] [Code Review] Added Force Encrypt option to iax.conf

Russell Bryant russell at digium.com
Thu Feb 12 13:42:52 CST 2009


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

Ship it!


Nice work, David!

- Russell


On 2009-02-12 10:46:55, David Vossel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/150/
> -----------------------------------------------------------
> 
> (Updated 2009-02-12 10:46:55)
> 
> 
> Review request for Asterisk Developers and Russell Bryant.
> 
> 
> Summary
> -------
> 
> There was a feature request to add any option into iax.conf to force encryption... so I made it. Basically if force encryption is enabled, encryption is enabled as well.  Encryption is checked during the processing of the AUTHREQ and AUTHREP frames.  If the encryption ie isn't present from the other side during this process, the call is terminated. When a call is placed to the box, the NEW frame contains the encryption method, so the authenticate_verify() function used during AUTHREP processing only checks to see if encryption method has been set rather than if the encryption ie is present.   When a call is placed from the box, the AUTHREQ message must contain the encryption ie in response to our NEW request or the call is terminated.  this check is in the authenticate_reply() function. 
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_iax2.c 174884 
> 
> Diff: http://reviewboard.digium.com/r/150/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David
> 
>




More information about the asterisk-dev mailing list