Merged revisions 89046 via svnmerge from
[asterisk/asterisk.git] / codecs / codec_zap.c
index 1b8a283..c38aa5a 100644 (file)
@@ -70,7 +70,7 @@ static struct channel_usage {
 static char *handle_cli_transcoder_show(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a);
 
 static struct ast_cli_entry cli[] = {
-       NEW_CLI(handle_cli_transcoder_show, "Display Zaptel transcoder utilization.")
+       AST_CLI_DEFINE(handle_cli_transcoder_show, "Display Zaptel transcoder utilization.")
 };
 
 struct format_map {
@@ -219,7 +219,6 @@ static void zap_destroy(struct ast_trans_pvt *pvt)
        if (ioctl(ztp->fd, ZT_TRANSCODE_OP, &x))
                ast_log(LOG_WARNING, "Failed to release transcoder channel: %s\n", strerror(errno));
 
-       ast_atomic_fetchadd_int(&channels.total, -1);
        switch (ztp->hdr->dstfmt) {
        case AST_FORMAT_G729A:
        case AST_FORMAT_G723_1:
@@ -281,7 +280,6 @@ static int zap_translate(struct ast_trans_pvt *pvt, int dest, int source)
        ztp->fd = fd;
        ztp->hdr = hdr;
 
-       ast_atomic_fetchadd_int(&channels.total, +1);
        switch (hdr->dstfmt) {
        case AST_FORMAT_G729A:
        case AST_FORMAT_G723_1:
@@ -380,14 +378,16 @@ static void unregister_translators(void)
        AST_LIST_UNLOCK(&translators);
 }
 
-static void parse_config(int reload)
+static int parse_config(int reload)
 {
        struct ast_variable *var;
        struct ast_flags config_flags = { reload ? CONFIG_FLAG_FILEUNCHANGED : 0 };
        struct ast_config *cfg = ast_config_load("codecs.conf", config_flags);
 
-       if (!cfg || cfg == CONFIG_STATUS_FILEUNCHANGED)
-               return;
+       if (!cfg)
+               return -1;
+       if (cfg == CONFIG_STATUS_FILEUNCHANGED)
+               return 0;
 
        for (var = ast_variable_browse(cfg, "plc"); var; var = var->next) {
               if (!strcasecmp(var->name, "genericplc")) {
@@ -396,8 +396,8 @@ static void parse_config(int reload)
                                           global_useplc ? "" : "not ");
               }
        }
-
        ast_config_destroy(cfg);
+       return 0;
 }
 
 static void build_translators(struct format_map *map, unsigned int dstfmts, unsigned int srcfmts)
@@ -436,6 +436,7 @@ static int find_transcoders(void)
        for (info.tcnum = 0; !(res = ioctl(fd, ZT_TRANSCODE_OP, &info)); info.tcnum++) {
                ast_verb(2, "Found transcoder '%s'.\n", info.name);
                build_translators(&map, info.dstfmts, info.srcfmts);
+               ast_atomic_fetchadd_int(&channels.total, info.numchannels / 2);
        }
        close(fd);
 
@@ -456,14 +457,15 @@ static int reload(void)
 {
        struct translator *cur;
 
-       parse_config(1);
+       if (parse_config(1))
+               return AST_MODULE_LOAD_DECLINE;
 
        AST_LIST_LOCK(&translators);
        AST_LIST_TRAVERSE(&translators, cur, entry)
                cur->t.useplc = global_useplc;
        AST_LIST_UNLOCK(&translators);
 
-       return 0;
+       return AST_MODULE_LOAD_SUCCESS;
 }
 
 static int unload_module(void)
@@ -476,11 +478,11 @@ static int unload_module(void)
 
 static int load_module(void)
 {
-       parse_config(0);
+       if (parse_config(0))
+               return AST_MODULE_LOAD_DECLINE;
        find_transcoders();
        ast_cli_register_multiple(cli, sizeof(cli) / sizeof(cli[0]));
-
-       return 0;
+       return AST_MODULE_LOAD_SUCCESS;
 }
 
 AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "Generic Zaptel Transcoder Codec Translator",