[asterisk-dev] [Code Review] The faxdetect option in chan_dahdi only works for the first FAX call.

rmudgett at digium.com rmudgett at digium.com
Wed Oct 13 15:19:06 CDT 2010

This is an automatically generated e-mail. To reply, visit:

(Updated 2010-10-13 15:19:06.273277)

Review request for Asterisk Developers and Russell Bryant.

Summary (updated)

The chan_dahdi faxdetect option only works for the first call.  After that 
the option no longer works.  The struct dahdi_pvt.callprogress member is 
the encoded user config setting for the callprogress and faxdetect config 
options.  Changing this value alters the configuration for all following 
calls until the chan_dahdi.conf file is reloaded.  

* Fixed the chan_dahdi ast_channel_setoption callback to not change the 
users faxdetect config setting except for the current call.  

* Fixed the chan_dahdi ast_channel_queryoption callback to read the active 
DSP setting of the faxdetect option.  

* Made actually disable the active faxdetect DSP setting for the current 
call on the analog port.  my_handle_dtmfup() is used for normal analog 
ports.  dahdi_handle_dtmfup() is the legacy code and is no longer used 
unless in a radio mode.  

This addresses bug 18116.


  /branches/1.8/channels/chan_dahdi.c 291540 

Diff: https://reviewboard.asterisk.org/r/972/diff


Compile and code inspection testing.



More information about the asterisk-dev mailing list