X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fpostgresql.c;h=d6dd0ba20e14399e30107aa1ef3f6674825c0ea3;hb=be99a844a40d7037a910d37a894988b807631a6e;hp=4e1c1e0c8127bebb693d71fc0b60116a84f1ad35;hpb=6597f3a6584704f92f824f3cf7bac3369102e8a0;p=collectd.git

diff --git a/src/postgresql.c b/src/postgresql.c
index 4e1c1e0c..d6dd0ba2 100644
--- a/src/postgresql.c
+++ b/src/postgresql.c
@@ -287,9 +287,9 @@ static int c_psql_exec_query (c_psql_database_t *db, udb_query_t *q)
 
 	int rows_num;
 	int status;
-	int i;
+	int row, col;
 
-	/* The user data may hold parameter information */
+	/* The user data may hold parameter information, but may be NULL. */
 	data = udb_query_get_user_data (q);
 
 	/* Versions up to `3' don't know how to handle parameters. */
@@ -340,12 +340,12 @@ static int c_psql_exec_query (c_psql_database_t *db, udb_query_t *q)
 		BAIL_OUT (-1);
 	}
 	
-	for (i = 0; i < column_num; i++) {
+	for (col = 0; col < column_num; ++col) {
 		/* Pointers returned by `PQfname' are freed by `PQclear' via
 		 * `BAIL_OUT'. */
-		column_names[i] = PQfname (res, i);
-		if (NULL == column_names[i]) {
-			log_err ("PQfname (%i) failed.", i);
+		column_names[col] = PQfname (res, col);
+		if (NULL == column_names[col]) {
+			log_err ("Failed to resolv name of column %i.", col);
 			BAIL_OUT (-1);
 		}
 	}
@@ -364,21 +364,20 @@ static int c_psql_exec_query (c_psql_database_t *db, udb_query_t *q)
 		BAIL_OUT (-1);
 	}
 
-	for (i = 0; i < rows_num; ++i) {
-		int j;
-
-		for (j = 0; j < column_num; j++) {
+	for (row = 0; row < rows_num; ++row) {
+		for (col = 0; col < column_num; ++col) {
 			/* Pointers returned by `PQgetvalue' are freed by `PQclear' via
 			 * `BAIL_OUT'. */
-			column_values[j] = PQgetvalue (res, /* row = */ i, /* col = */ j);
-			if (NULL == column_values[j]) {
-				log_err ("PQgetvalue (%i, %i) failed.", i, j);
+			column_values[col] = PQgetvalue (res, row, col);
+			if (NULL == column_values[col]) {
+				log_err ("Failed to get value at (row = %i, col = %i).",
+						row, col);
 				break;
 			}
 		}
 
 		/* check for an error */
-		if (j < column_num)
+		if (col < column_num)
 			continue;
 
 		status = udb_query_handle_result (q, column_values);
@@ -386,7 +385,7 @@ static int c_psql_exec_query (c_psql_database_t *db, udb_query_t *q)
 			log_err ("udb_query_handle_result failed with status %i.",
 					status);
 		}
-	} /* for (i = 0; i < rows_num; ++i) */
+	} /* for (row = 0; row < rows_num; ++row) */
 
 	BAIL_OUT (0);
 #undef BAIL_OUT
@@ -631,6 +630,12 @@ static int c_psql_config_database (oconfig_item_t *ci)
 					&db->queries, &db->queries_num);
 	}
 
+	for (i = 0; (size_t)i < db->queries_num; ++i) {
+		c_psql_user_data_t *data;
+		data = udb_query_get_user_data (db->queries[i]);
+		if ((data != NULL) && (data->params_num > db->max_params_num))
+			db->max_params_num = data->params_num;
+	}
 	return 0;
 } /* c_psql_config_database */