[asterisk-commits] trunk r30194 - /trunk/apps/app_queue.c

[asterisk-commits at lists.digium.com]

Author: bweschke
Date: Thu May 25 08:51:44 2006
New Revision: 30194

URL: http://svn.digium.com/view/asterisk?rev=30194&view=rev
Log:
 Making sure a char ptr is initialized before we strchr on it is a GOOD thing. Ya! Testing! 


Modified:
    trunk/apps/app_queue.c

Modified: trunk/apps/app_queue.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_queue.c?rev=30194&r1=30193&r2=30194&view=diff
==============================================================================
--- trunk/apps/app_queue.c (original)
+++ trunk/apps/app_queue.c Thu May 25 08:51:44 2006
@@ -2389,14 +2389,15 @@
 					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))