[asterisk-bugs] [Asterisk 0014728]: [patch] global mohinterpret setting is ignored

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14728 
====================================================================== 
Reported By:                dimas
Assigned To:                dvossel
====================================================================== 
Project:                    Asterisk
Issue ID:                   14728
Category:                   Channels/chan_iax2
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 166771 
Request Review:              
====================================================================== 
Date Submitted:             2009-03-23 17:20 CDT
Last Modified:              2009-05-04 17:13 CDT
====================================================================== 
Summary:                    [patch] global mohinterpret setting is ignored
Description: 
iax.conf says that:

; This option may be specified globally, or on a per-user or per-peer
basis.

in fact, when option is specified globally (in the [general] section), its
value is ignored. I believe this is because global value does not get
copied into pvt structure on build_peer.
====================================================================== 

---------------------------------------------------------------------- 
 (0104164) dvossel (administrator) - 2009-05-04 17:13
 http://bugs.digium.com/view.php?id=14728#c104164 
---------------------------------------------------------------------- 
chan_iax.c line 1742. global mohinterpret is copied to pvt.  in
check_access() line 5449. mohinterpret from user is copied into pvt if it
exists.  

---------------
*CLI> [May  4 17:00:31] NOTICE[28123]: chan_iax2.c:1748 __find_callno:
COPY mohinterpret global :moh_test to pvt
    -- Accepted AUTHENTICATED TBD call from 10.24.20.218
    -- Accepting DIAL from 10.24.20.218, formats = 0x4
    -- Executing [6001 at local:1] Dial("IAX2/iaxy-7208", "SIP/6001") in new
stack
    -- Called 6001
    -- SIP/6001-010caf10 is ringing
[May  4 17:00:34] NOTICE[28140]: chan_iax2.c:3869 iax2_indicate:
pvt->mohinterpret=moh_test
    -- SIP/6001-010caf10 answered IAX2/iaxy-7208
[May  4 17:00:35] NOTICE[28140]: chan_iax2.c:3869 iax2_indicate:
pvt->mohinterpret=moh_test
[May  4 17:00:35] NOTICE[28140]: chan_iax2.c:3869 iax2_indicate:
pvt->mohinterpret=moh_test
[May  4 17:00:35] NOTICE[28140]: chan_iax2.c:3869 iax2_indicate:
pvt->mohinterpret=moh_test
[May  4 17:00:36] NOTICE[28118]: chan_sip.c:16610 sip_poke_noanswer: Peer
'SIP_cupbear' is now UNREACHABLE!  Last qualify: 0
[May  4 17:00:38] NOTICE[28140]: chan_iax2.c:3869 iax2_indicate:
pvt->mohinterpret=moh_test
    -- Started music on hold, class 'moh_test', on IAX2/iaxy-7208
    -- Stopped music on hold on IAX2/iaxy-7208
  == Spawn extension (local, 6001, 1) exited non-zero on 'IAX2/iaxy-7208'
    -- Hungup 'IAX2/iaxy-7208'
------------------------

In iax.conf, globaly mohinterpret=moh_test. The friend has no mohinterpret
set. global mohinterpret "moh_test" is copied into the pvt, works as
expected for me (as seen in cli output).  I tried global mohinterpret =
passthrough as well, worked as expected. 

[May  4 17:10:36] NOTICE[29101]: chan_iax2.c:3846 iax2_indicate:
pvt->mohinterpret=passthrough

This was tested in both latest 1.4 branch code and the 1.4.24.1 tag.  I
really don't know what I else I can do to resolve this.  Is there anything
I'm missing? 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-05-04 17:13 dvossel        Note Added: 0104164                          
======================================================================