[asterisk-bugs] [Asterisk 0018044]: [patch] Crash in check_sounds

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18044 
====================================================================== 
Reported By:                murf
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18044
Category:                   Sounds
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.2.14-rc1 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-09-23 23:37 CDT
Last Modified:              2010-09-23 23:37 CDT
====================================================================== 
Summary:                    [patch] Crash in check_sounds
Description: 
When I demo'd this for an interested party, it crashed!
So I took some time and debugged it. Simply, I got confused
on which variable to use as the file count. I was using numents,
but should have been using 'ind'. So I fixed it, and added an 
extra printf to say how many actual files were in the list (it
does tell you how many entries are in the script file.). No more
crashes on stock script files issued for 1.4.19 (core).

Now, I also added code so it will ignore comment lines that start with a
';' in the
first char of the line.

I suggest we mod the script files to follow the above format for 
comments. It will make running the checks easier.

====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-09-23 23:37 murf           New Issue                                    
2010-09-23 23:37 murf           Asterisk Version          => 1.6.2.14-rc1    
2010-09-23 23:37 murf           Regression                => No              
2010-09-23 23:37 murf           SVN Branch (only for SVN checkouts, not tarball
releases) => N/A             
======================================================================