diff options
author | BJ Weschke <bweschke@btwtech.com> | 2006-05-25 13:51:44 +0000 |
---|---|---|
committer | BJ Weschke <bweschke@btwtech.com> | 2006-05-25 13:51:44 +0000 |
commit | 36cf242f31f2d71e8551ffaa683f23e31252431f (patch) | |
tree | 96d174cb10813f2f2e329002c702a7c37fc470dc /apps | |
parent | 8d890cb62663fd2e7620931b6c7366a3615a3445 (diff) |
Making sure a char ptr is initialized before we strchr on it is a GOOD thing. Ya! Testing!
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@30194 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'apps')
-rw-r--r-- | apps/app_queue.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/apps/app_queue.c b/apps/app_queue.c index 9b75d11d0..d42f20e85 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -2389,14 +2389,15 @@ static int try_calling(struct queue_ent *qe, const char *options, char *announce ast_log(LOG_WARNING, "monitor-format (in queues.conf) and MONITOR_FILENAME cannot contain a '|'! Not recording.\n"); mixmonapp = NULL; } + + if (!monitor_options) + monitor_options = ast_strdupa(""); if (strchr(monitor_options, '|')) { ast_log(LOG_WARNING, "MONITOR_OPTIONS cannot contain a '|'! Not recording.\n"); mixmonapp = NULL; } - if (!monitor_options) - monitor_options = ast_strdupa(""); if (mixmonapp) { if (!ast_strlen_zero(monitor_exec) && !ast_strlen_zero(monitor_options)) |