diff options
author | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-04-10 01:51:47 +0000 |
---|---|---|
committer | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-04-10 01:51:47 +0000 |
commit | 18a8186d32282181df25fa1aeb0c276185009a3f (patch) | |
tree | 77944d35f1cef59a64cdb19acf7896f18a98bc00 /res | |
parent | c40963221ede37f1ffac1e4691b83d96986b3200 (diff) |
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: http://svn.digium.com/svn/asterisk/trunk@18693 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'res')
-rw-r--r-- | res/res_config_pgsql.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/res/res_config_pgsql.c b/res/res_config_pgsql.c index 114fcfb3f..8c4d63c33 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); |