]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
automerge commit
authorAutomerge Script <automerge@asterisk.org>
Thu, 21 Jun 2007 19:13:44 +0000 (19:13 +0000)
committerAutomerge Script <automerge@asterisk.org>
Thu, 21 Jun 2007 19:13:44 +0000 (19:13 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.2-netsec@70805 65c4cc65-6c06-0410-ace0-fbb531ad65f3

cdr/cdr_custom.c

index 64fddb368201b4a7f3f8fd01730206c24de7ad61..dfd98388e0ad341fc76187119ad91fc56a0b424e 100644 (file)
@@ -72,21 +72,22 @@ static int load_config(int reload)
 
        strcpy(format, "");
        strcpy(master, "");
+       ast_mutex_lock(&lock);
        if((cfg = ast_config_load("cdr_custom.conf"))) {
                var = ast_variable_browse(cfg, "mappings");
                while(var) {
-                       ast_mutex_lock(&lock);
                        if (!ast_strlen_zero(var->name) && !ast_strlen_zero(var->value)) {
                                if (strlen(var->value) > (sizeof(format) - 2))
                                        ast_log(LOG_WARNING, "Format string too long, will be truncated, at line %d\n", var->lineno);
                                strncpy(format, var->value, sizeof(format) - 2);
                                strcat(format,"\n");
                                snprintf(master, sizeof(master),"%s/%s/%s", ast_config_AST_LOG_DIR, name, var->name);
-                               ast_mutex_unlock(&lock);
+                               if (var->next) {
+                                       ast_log(LOG_NOTICE, "Sorry, only one mapping is supported at this time, mapping '%s' will be ignored at line %d.\n", var->next->name, var->next->lineno); 
+                                       break;
+                               }
                        } else
                                ast_log(LOG_NOTICE, "Mapping must have both filename and format at line %d\n", var->lineno);
-                       if (var->next)
-                               ast_log(LOG_NOTICE, "Sorry, only one mapping is supported at this time, mapping '%s' will be ignored at line %d.\n", var->next->name, var->next->lineno); 
                        var = var->next;
                }
                ast_config_destroy(cfg);
@@ -97,6 +98,7 @@ static int load_config(int reload)
                else
                        ast_log(LOG_WARNING, "Failed to load configuration file. Module not activated.\n");
        }
+       ast_mutex_unlock(&lock);
        
        return res;
 }