[asterisk-bugs] [Asterisk 0018457]: [patch] moh reload leaks file descriptors to dahdi timing channel

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18457 
====================================================================== 
Reported By:                mcallist
Assigned To:                qwell
====================================================================== 
Project:                    Asterisk
Issue ID:                   18457
Category:                   Resources/res_musiconhold
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Target Version:             1.4.41
Asterisk Version:           SVN 
JIRA:                       SWP-2744 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-12-10 15:17 CST
Last Modified:              2011-02-01 11:02 CST
====================================================================== 
Summary:                    [patch] moh reload leaks file descriptors to dahdi
timing channel
Description: 
Issuing an moh reload or even unload/load on res_musiconhold.so causes
DAHDI_FILE_PSEUDO to be opened, and the old reference to the open file to
be leaked.

This eventually can lead to exhaustion of the DAHDI channels and locking
out of meetme usage.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0018262 [patch] No MOH When Call Parked
====================================================================== 

---------------------------------------------------------------------- 
 (0131353) svnbot (reporter) - 2011-02-01 11:02
 https://issues.asterisk.org/view.php?id=18457#c131353 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 305472

_U  branches/1.6.2/
U   branches/1.6.2/res/res_musiconhold.c

------------------------------------------------------------------------
r305472 | qwell | 2011-02-01 11:02:09 -0600 (Tue, 01 Feb 2011) | 16 lines

Merged revisions 305471 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r305471 | qwell | 2011-02-01 11:00:55 -0600 (Tue, 01 Feb 2011) | 9 lines
  
  Close file descriptor for timing source when a MOH class gets destroyed.
  
  (closes issue https://issues.asterisk.org/view.php?id=18457)
  Reported by: mcallist
  Patches: 
        18457-closetimer.diff uploaded by qwell (license 4)
        18457-closetimer_trunk.diff uploaded by qwell (license 4)
  Tested by: qwell, loloski
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=305472 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-02-01 11:02 svnbot         Note Added: 0131353                          
======================================================================