[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:05 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
====================================================================== 

---------------------------------------------------------------------- 
 (0131355) svnbot (reporter) - 2011-02-01 11:05
 https://issues.asterisk.org/view.php?id=18457#c131355 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 305474

_U  trunk/
U   trunk/res/res_musiconhold.c

------------------------------------------------------------------------
r305474 | qwell | 2011-02-01 11:05:39 -0600 (Tue, 01 Feb 2011) | 30 lines

Merged revisions 305473 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r305473 | qwell | 2011-02-01 11:04:23 -0600 (Tue, 01 Feb 2011) | 23
lines
  
  Merged revisions 305472 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2
  
  ................
    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=305474 

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