]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Fixes unitialized variable use warning introduced by addition of mixmonitor forward...
authorJonathan Rose <jrose@digium.com>
Thu, 8 Mar 2012 18:40:13 +0000 (18:40 +0000)
committerJonathan Rose <jrose@digium.com>
Thu, 8 Mar 2012 18:40:13 +0000 (18:40 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8-digiumphones@358725 65c4cc65-6c06-0410-ace0-fbb531ad65f3

apps/app_mixmonitor.c

index 0705502304a3e07e328fff9a9c6e7a6929a84d65..7176de91a0016b5c10ba5327d8b4ab80dcb3345e 100644 (file)
@@ -424,7 +424,7 @@ static void *mixmonitor_thread(void *obj)
        struct mixmonitor *mixmonitor = obj;
        struct ast_filestream **fs = NULL;
        unsigned int oflags;
-       char *ext;
+       char *ext = "";
        char *last_slash;
        int errflag = 0;
 
@@ -507,8 +507,13 @@ static void *mixmonitor_thread(void *obj)
        ast_verb(2, "End MixMonitor Recording %s\n", mixmonitor->name);
 
        if (!AST_LIST_EMPTY(&mixmonitor->recipient_list)) {
-               ast_verb(3, "Copying recordings for Mixmonitor %s to voicemail recipients\n", mixmonitor->name);
-               copy_to_voicemail(mixmonitor, ext);
+               if (ast_strlen_zero(ext)) {
+                       ast_log(LOG_ERROR, "No file extension set for Mixmonitor %s. Skipping copy to voicemail.\n",
+                               mixmonitor -> name);
+               } else {
+                       ast_verb(3, "Copying recordings for Mixmonitor %s to voicemail recipients\n", mixmonitor->name);
+                       copy_to_voicemail(mixmonitor, ext);
+               }
        } else {
                ast_debug(3, "No recipients to forward monitor to, moving on.\n");
        }