kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-31 02:37:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

res_monitor.c: Don't emit a warning about 'X' being unrecognized.

Browse Source 
Code was added in 030f7d41 to warn if an unrecognized option was
passed to an application, but code in Monitor was taking advantage of
the fact that the application would silently accept an invalid option.

We now recognize the invalid option but we don't do anything if it's
set.

Fixes #639

(cherry picked from commit 3c1a2daecd)
This commit is contained in:
Sean Bright
2024-03-07 12:27:12 -05:00
committed by Asterisk Development Team
parent 4cba518447
commit 4174709f7d
1 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
res/res_monitor.c
View File

@@ -671,6 +671,19 @@ enum {
MON_FLAG_DROP_IN = (1 << 2),
MON_FLAG_DROP_OUT = (1 << 3),
MON_FLAG_BEEP = (1 << 4),
/*!
* \brief Dummy option to silence WARNINGs.
*
* Application argument processing was updated to emit a WARNING when an
* unrecognized option was encountered. Previously, unrecognized options
* were silently ignored which was taken advantage of by code in this module
* to "stamp out" problematic application arguments and replace them with 'X.'
*
* So to silence the warning we add 'X' as a valid option and just don't do
* anything if it's set.
*/
MON_FLAG_DUMMY = (1 << 5),
};
enum {
@@ -684,6 +697,7 @@ AST_APP_OPTIONS(monitor_opts, {
AST_APP_OPTION('i', MON_FLAG_DROP_IN),
AST_APP_OPTION('o', MON_FLAG_DROP_OUT),
AST_APP_OPTION_ARG('B', MON_FLAG_BEEP, OPT_ARG_BEEP_INTERVAL),
AST_APP_OPTION('X', MON_FLAG_DUMMY),
});
/*!