From: Russell Bryant Date: Mon, 10 Apr 2006 01:51:47 +0000 (+0000) Subject: add back braces that were removed in a recent commit that leave a bunch of X-Git-Tag: 1.4.0-beta1~2068 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=d203e9c6c03a4ab34648582fb88256fc50768a96;p=thirdparty%2Fasterisk.git add back braces that were removed in a recent commit that leave a bunch of nested statements in a block without branches, which is a violation of the coding guidelines. As a matter of fact, this module violates the coding guidelines in multiple ways that including formatting and code issues. In my opinion, this module should not have been merged into the trunk in this form. git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18693 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- diff --git a/res/res_config_pgsql.c b/res/res_config_pgsql.c index 114fcfb3fa..8c4d63c33c 100644 --- a/res/res_config_pgsql.c +++ b/res/res_config_pgsql.c @@ -159,7 +159,7 @@ static struct ast_variable *realtime_pgsql(const char *database, const char *tab } for(i = 0; i < numFields; i++) fieldnames[i]=PQfname(result,i); - for(rowIndex = 0; rowIndex < num_rows; rowIndex++) + for(rowIndex = 0; rowIndex < num_rows; rowIndex++) { for(i = 0; i < numFields; i++) { stringp = PQgetvalue(result,rowIndex,i); while(stringp) { @@ -176,6 +176,7 @@ static struct ast_variable *realtime_pgsql(const char *database, const char *tab } } } + } free(fieldnames); } else { ast_log(LOG_WARNING, "Postgresql RealTime: Could not find any rows in table %s.\n", table);