diff --git a/res/res_config_odbc.c b/res/res_config_odbc.c index 86cc25c86e..fc2b8fa1b9 100644 --- a/res/res_config_odbc.c +++ b/res/res_config_odbc.c @@ -419,7 +419,7 @@ static struct ast_config *realtime_multi_odbc(const char *database, const char * if ((res != SQL_SUCCESS) && (res != SQL_SUCCESS_WITH_INFO)) { ast_log(LOG_WARNING, "SQL Describe Column error!\n[%s]\n\n", sql); ast_category_destroy(cat); - continue; + goto next_sql_fetch; } indicator = 0; @@ -430,7 +430,7 @@ static struct ast_config *realtime_multi_odbc(const char *database, const char * if ((res != SQL_SUCCESS) && (res != SQL_SUCCESS_WITH_INFO)) { ast_log(LOG_WARNING, "SQL Get Data error!\n[%s]\n\n", sql); ast_category_destroy(cat); - continue; + goto next_sql_fetch; } stringp = rowdata; while (stringp) { @@ -448,6 +448,7 @@ static struct ast_config *realtime_multi_odbc(const char *database, const char * } } ast_category_append(cfg, cat); +next_sql_fetch:; } SQLFreeHandle(SQL_HANDLE_STMT, stmt); diff --git a/res/res_config_pgsql.c b/res/res_config_pgsql.c index 6ed6bd294f..f8de865963 100644 --- a/res/res_config_pgsql.c +++ b/res/res_config_pgsql.c @@ -566,6 +566,7 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char PQfinish(pgsqlConn); pgsqlConn = NULL; } + ast_config_destroy(cfg); return NULL; } @@ -586,6 +587,7 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char if (pgresult) { ast_log(LOG_ERROR, "PostgreSQL RealTime: detected invalid input: '%s'\n", newval); va_end(ap); + ast_config_destroy(cfg); return NULL; } @@ -601,6 +603,7 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char if (pgresult) { ast_log(LOG_ERROR, "PostgreSQL RealTime: detected invalid input: '%s'\n", newval); va_end(ap); + ast_config_destroy(cfg); return NULL; } @@ -616,8 +619,9 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char /* We now have our complete statement; Lets connect to the server and execute it. */ ast_mutex_lock(&pgsql_lock); - if (pgsql_exec(database, table, ast_str_buffer(sql), &result) != 0) { + if (pgsql_exec(database, table, ast_str_buffer(sql), &result) != 0) { ast_mutex_unlock(&pgsql_lock); + ast_config_destroy(cfg); return NULL; } else { ExecStatusType result_status = PQresultStatus(result); @@ -631,6 +635,7 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char PQresultErrorMessage(result), PQresStatus(result_status)); PQclear(result); ast_mutex_unlock(&pgsql_lock); + ast_config_destroy(cfg); return NULL; } } @@ -648,6 +653,7 @@ static struct ast_config *realtime_multi_pgsql(const char *database, const char if (!(fieldnames = ast_calloc(1, numFields * sizeof(char *)))) { PQclear(result); ast_mutex_unlock(&pgsql_lock); + ast_config_destroy(cfg); return NULL; } for (i = 0; i < numFields; i++)