[asterisk-bugs] [Asterisk 0019134]: [patch] Unclear code in app_dial.c
Asterisk Bug Tracker
noreply at bugs.digium.com
Mon Apr 18 09:07:52 CDT 2011
The following issue has been UPDATED.
======================================================================
https://issues.asterisk.org/view.php?id=19134
======================================================================
Reported By: oej
Assigned To:
======================================================================
Project: Asterisk
Issue ID: 19134
Category: Applications/app_dial
Reproducibility: always
Severity: minor
Priority: normal
Status: new
Asterisk Version: 1.8.2.4
JIRA:
Regression: No
Reviewboard Link:
SVN Branch (only for SVN checkouts, not tarball releases): N/A
SVN Revision (number only!):
Request Review:
======================================================================
Date Submitted: 2011-04-18 09:03 CDT
Last Modified: 2011-04-18 09:07 CDT
======================================================================
Summary: [patch] Unclear code in app_dial.c
Description:
The code that handles AST_CONTROL_PROGRESS in the function wait_for_answer
has unclear formatting. The line checking for OPT_RINGBACK has no curly
brackets, so it only applies to the next level if. There's a second if()
statement where indents indicate that it should be part of the RINGBACK
clause, but it is not. I can't say what's righ there, so someone else needs
to check the code and decide. Thanks.
======================================================================
Issue History
Date Modified Username Field Change
======================================================================
2011-04-18 09:07 oej Summary Unclear code in
app_dial.c => [patch] Unclear code in app_dial.c
======================================================================
More information about the asterisk-bugs
mailing list