Merged revisions 74515 via svnmerge from
[asterisk/asterisk.git] / cdr / cdr_csv.c
index 3ce99ec..f40f555 100644 (file)
  * \ingroup cdr_drivers
  */
 
+#include "asterisk.h"
+
+ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
+
 #include <sys/types.h>
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>
 #include <time.h>
 
-#include "asterisk.h"
-
-ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
-
 #include "asterisk/config.h"
 #include "asterisk/channel.h"
 #include "asterisk/cdr.h"
 #include "asterisk/module.h"
+#include "asterisk/options.h"
 #include "asterisk/logger.h"
 #include "asterisk/utils.h"
 
@@ -89,8 +90,6 @@ static char *config = "cdr.conf";
   "userfield"          user field set via SetCDRUserField 
 ----------------------------------------------------------*/
 
-static char *desc = "Comma Separated Values CDR Backend";
-
 static char *name = "csv";
 
 static FILE *mf = NULL;
@@ -100,7 +99,7 @@ static int load_config(void)
 {
        struct ast_config *cfg;
        struct ast_variable *var;
-       char *tmp;
+       const char *tmp;
 
        usegmtime = 0;
        loguniqueid = 0;
@@ -110,20 +109,20 @@ static int load_config(void)
        
        if (!cfg) {
                ast_log(LOG_WARNING, "unable to load config: %s\n", config);
-               return -1;
+               return 0;
        } 
        
        var = ast_variable_browse(cfg, "csv");
        if (!var) {
                ast_config_destroy(cfg);
-               return -1;
+               return 0;
        }
        
        tmp = ast_variable_retrieve(cfg, "csv", "usegmtime");
        if (tmp) {
                usegmtime = ast_true(tmp);
                if (usegmtime) {
-                       ast_log(LOG_DEBUG, "logging time in GMT\n");
+                       ast_debug(1, "logging time in GMT\n");
                }
        }
 
@@ -131,7 +130,7 @@ static int load_config(void)
        if (tmp) {
                loguniqueid = ast_true(tmp);
                if (loguniqueid) {
-                       ast_log(LOG_DEBUG, "logging CDR field UNIQUEID\n");
+                       ast_debug(1, "logging CDR field UNIQUEID\n");
                }
        }
 
@@ -139,12 +138,12 @@ static int load_config(void)
        if (tmp) {
                loguserfield = ast_true(tmp);
                if (loguserfield) {
-                       ast_log(LOG_DEBUG, "logging CDR user-defined field\n");
+                       ast_debug(1, "logging CDR user-defined field\n");
                }
        }
 
        ast_config_destroy(cfg);
-       return 0;
+       return 1;
 }
 
 static int append_string(char *buf, char *s, size_t bufsize)
@@ -201,7 +200,7 @@ static int append_date(char *buf, struct timeval tv, size_t bufsize)
        if (usegmtime) {
                gmtime_r(&t,&tm);
        } else {
-               localtime_r(&t,&tm);
+               ast_localtime(&t, &tm, NULL);
        }
        strftime(tmp, sizeof(tmp), DATE_FORMAT, &tm);
        return append_string(buf, tmp, bufsize);
@@ -261,7 +260,7 @@ static int build_csv_record(char *buf, size_t bufsize, struct ast_cdr *cdr)
 
 static int writefile(char *s, char *acc)
 {
-       char tmp[AST_CONFIG_MAX_PATH];
+       char tmp[PATH_MAX];
        FILE *f;
        if (strchr(acc, '/') || (acc[0] == '.')) {
                ast_log(LOG_WARNING, "Account code '%s' insecure for writing file\n", acc);
@@ -282,7 +281,7 @@ static int csv_log(struct ast_cdr *cdr)
 {
        /* Make sure we have a big enough buf */
        char buf[1024];
-       char csvmaster[AST_CONFIG_MAX_PATH];
+       char csvmaster[PATH_MAX];
        snprintf(csvmaster, sizeof(csvmaster),"%s/%s/%s", ast_config_AST_LOG_DIR, CSV_LOG_DIR, CSV_MASTER);
 #if 0
        printf("[CDR] %s ('%s' -> '%s') Dur: %ds Bill: %ds Disp: %s Flags: %s Account: [%s]\n", cdr->channel, cdr->src, cdr->dst, cdr->duration, cdr->billsec, ast_cdr_disp2str(cdr->disposition), ast_cdr_flags2str(cdr->amaflags), cdr->accountcode);
@@ -311,12 +310,7 @@ static int csv_log(struct ast_cdr *cdr)
        return 0;
 }
 
-const char *description(void)
-{
-       return desc;
-}
-
-int unload_module(void)
+static int unload_module(void)
 {
        if (mf)
                fclose(mf);
@@ -324,13 +318,14 @@ int unload_module(void)
        return 0;
 }
 
-int load_module(void)
+static int load_module(void)
 {
        int res;
        
-       load_config();
+       if(!load_config())
+               return AST_MODULE_LOAD_DECLINE;
 
-       res = ast_cdr_register(name, desc, csv_log);
+       res = ast_cdr_register(name, ast_module_info->description, csv_log);
        if (res) {
                ast_log(LOG_ERROR, "Unable to register CSV CDR handling\n");
                if (mf)
@@ -339,18 +334,15 @@ int load_module(void)
        return res;
 }
 
-int reload(void)
+static int reload(void)
 {
        load_config();
-       return 0;
-}
 
-int usecount(void)
-{
        return 0;
 }
 
-const char *key()
-{
-       return ASTERISK_GPL_KEY;
-}
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "Comma Separated Values CDR Backend",
+               .load = load_module,
+               .unload = unload_module,
+               .reload = reload,
+              );