[asterisk-bugs] [Asterisk 0016993]: [patch] MusicOnHold produces a crash

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16993 
====================================================================== 
Reported By:                dmitri
Assigned To:                dvossel
====================================================================== 
Project:                    Asterisk
Issue ID:                   16993
Category:                   Resources/res_musiconhold
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
JIRA:                       SWP-1062 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 251399 
Request Review:              
====================================================================== 
Date Submitted:             2010-03-09 02:29 CST
Last Modified:              2010-06-04 17:52 CDT
====================================================================== 
Summary:                    [patch] MusicOnHold produces a crash
Description: 
MusicOnHold produces a crash on hangup when class configured in realtime
refers for non existing or an empty directory.
And this only happens when using cachertclasses flag in musiconhold.conf.
I made a patch for fixing this issue.
====================================================================== 

---------------------------------------------------------------------- 
 (0122997) dvossel (administrator) - 2010-06-04 17:52
 https://issues.asterisk.org/view.php?id=16993#c122997 
---------------------------------------------------------------------- 
I am unfamiliar with this code.  Can you explain exactly why your original
patch resolve this crash.  It would be nice to see a back trace of what
this crash looks like so I can better understand what is being resolved. 
Just looking at your patch alone it is not obvious to me what that is
supposed to fix.  Thanks for your feedback! 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-06-04 17:52 dvossel        Note Added: 0122997                          
======================================================================