[asterisk-bugs] [Asterisk 0018044]: [patch] Crash in check_sounds
Asterisk Bug Tracker
noreply at bugs.digium.com
Mon Sep 27 14:24:12 CDT 2010
The following issue is now READY FOR TESTING.
======================================================================
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: ready for testing
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-27 14:24 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.
======================================================================
Relationships ID Summary
----------------------------------------------------------------------
related to 0017473 [patch] mistakes in the 1.4.19 core-sou...
======================================================================
Issue History
Date Modified Username Field Change
======================================================================
2010-09-27 14:24 lmadsen Status new => ready for
testing
======================================================================
More information about the asterisk-bugs
mailing list