diff options
| author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-07-27 22:23:38 +0000 |
|---|---|---|
| committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-07-27 22:23:38 +0000 |
| commit | c4d69251f5a82094b73987955f98ec3bfe24c0af (patch) | |
| tree | eee648944e90378b43a37e3362f1e6096ed8fdc5 | |
| parent | 20a7edaed06e844ac967cffdcd01ba231e7bd29c (diff) | |
use the enum that defines the option arguments, so that the likelihood of mismatched option indexes is reduced (which in this case was a bug, the volume argument was not checked properly)
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.2@38370 f38db490-d61c-443f-a65b-d21fe96a405b
| -rw-r--r-- | apps/app_chanspy.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/apps/app_chanspy.c b/apps/app_chanspy.c index 1c8851750..90a8574dd 100644 --- a/apps/app_chanspy.c +++ b/apps/app_chanspy.c @@ -419,19 +419,19 @@ static int chanspy_exec(struct ast_channel *chan, void *data) char *opts[OPT_ARG_ARRAY_SIZE]; ast_app_parse_options(chanspy_opts, &flags, opts, options); if (ast_test_flag(&flags, OPTION_GROUP)) { - mygroup = opts[1]; + mygroup = opts[OPT_ARG_GROUP]; } if (ast_test_flag(&flags, OPTION_RECORD)) { - if (!(recbase = opts[2])) { + if (!(recbase = opts[OPT_ARG_RECORD])) { recbase = "chanspy"; } } silent = ast_test_flag(&flags, OPTION_QUIET); bronly = ast_test_flag(&flags, OPTION_BRIDGED); - if (ast_test_flag(&flags, OPTION_VOLUME) && opts[1]) { + if (ast_test_flag(&flags, OPTION_VOLUME) && opts[OPT_ARG_VOLUME]) { int vol; - if ((sscanf(opts[0], "%d", &vol) != 1) || (vol > 4) || (vol < -4)) + if ((sscanf(opts[OPT_ARG_VOLUME], "%d", &vol) != 1) || (vol > 4) || (vol < -4)) ast_log(LOG_NOTICE, "Volume factor must be a number between -4 and 4\n"); else volfactor = vol; |