Merged revisions 29196 via svnmerge from
[asterisk/asterisk.git] / res / res_config_pgsql.c
index 85e8945..2e900d0 100644 (file)
  * \arg http://www.postgresql.org
  */
 
+/*** MODULEINFO
+       <depend>pgsql</depend>
+ ***/
+
 #include <stdlib.h>
 #include <string.h>
 #include <libpq-fe.h>                  /* PostgreSQL */
@@ -41,8 +45,6 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #include "asterisk/utils.h"
 #include "asterisk/cli.h"
 
-static char *res_config_pgsql_desc = "Postgresql RealTime Configuration Driver";
-
 AST_MUTEX_DEFINE_STATIC(pgsql_lock);
 
 #define RES_CONFIG_PGSQL_CONF "res_pgsql.conf"
@@ -237,7 +239,7 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char
        }
 
        initfield = ast_strdupa(newparam);
-       if (initfield && (op = strchr(initfield, ' '))) {
+       if ((op = strchr(initfield, ' '))) {
                *op = '\0';
        }
 
@@ -317,11 +319,8 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char
 
                for (rowIndex = 0; rowIndex < num_rows; rowIndex++) {
                        var = NULL;
-                       cat = ast_category_new("");
-                       if (!cat) {
-                               ast_log(LOG_WARNING, "Out of memory!\n");
+                       if (!(cat = ast_category_new("")))
                                continue;
-                       }
                        for (i = 0; i < numFields; i++) {
                                stringp = PQgetvalue(result, rowIndex, i);
                                while (stringp) {
@@ -524,10 +523,8 @@ static struct ast_config *config_pgsql(const char *database, const char *table,
 
                        if (strcmp(last, field_category) || last_cat_metric != atoi(field_cat_metric)) {
                                cur_cat = ast_category_new(field_category);
-                               if (!cur_cat) {
-                                       ast_log(LOG_WARNING, "Out of memory!\n");
+                               if (!cur_cat)
                                        break;
-                               }
                                strcpy(last, field_category);
                                last_cat_metric = atoi(field_cat_metric);
                                ast_category_append(cfg, cur_cat);
@@ -554,7 +551,7 @@ static struct ast_config_engine pgsql_engine = {
        .update_func = update_pgsql
 };
 
-int load_module(void)
+static int load_module(void *mod)
 {
        parse_config();
 
@@ -578,7 +575,7 @@ int load_module(void)
        return 0;
 }
 
-int unload_module(void)
+static int unload_module(void *mod)
 {
        /* Aquire control before doing anything to the module itself. */
        ast_mutex_lock(&pgsql_lock);
@@ -601,7 +598,7 @@ int unload_module(void)
        return 0;
 }
 
-int reload(void)
+static int reload(void *mod)
 {
        /* Aquire control before doing anything to the module itself. */
        ast_mutex_lock(&pgsql_lock);
@@ -698,13 +695,15 @@ int parse_config(void)
        return 1;
 }
 
-const char *description(void)
+static const char *description(void)
 {
-       return res_config_pgsql_desc;
+       return "Postgresql RealTime Configuration Driver";
+
 }
 
-int usecount(void)
+static int usecount(void)
 {
+       /* XXX check this... */
        /* Try and get a lock. If unsuccessful, than that means another thread is using the pgsql object. */
        if (ast_mutex_trylock(&pgsql_lock)) {
                ast_log(LOG_DEBUG, "Postgresql RealTime: Module usage count is 1.\n");
@@ -714,11 +713,13 @@ int usecount(void)
        return 0;
 }
 
-const char *key()
+static const char *key(void)
 {
        return ASTERISK_GPL_KEY;
 }
 
+STD_MOD(MOD_0, reload, NULL, NULL);
+
 static int pgsql_reconnect(const char *database)
 {
        char my_database[50];