[libgda/LIBGDA_4.0] More corrections after running cppcheck

[Vivien Malerba <vivien src gnome org>]

commit 7f906fb522b1e2575561efbc021998cd7a275318
Author: Vivien Malerba <malerba gnome-db org>
Date:   Sun Nov 28 19:10:25 2010 +0100

    More corrections after running cppcheck

 libgda/binreloc/gda-binreloc.c                   |    4 +++-
 libgda/gda-server-provider-extra.c               |    2 +-
 libgda/gda-util.c                                |    4 ----
 libgda/sql-parser/parser.y                       |    2 +-
 libgda/sqlite/virtual/gda-vprovider-data-model.c |    2 +-
 providers/firebird/parser.y                      |    2 +-
 providers/skel-implementation/capi/parser.y      |    2 +-
 samples/XSLT/transform.c                         |    2 +-
 tests/meta-store/common.c                        |    3 +--
 tests/test-cnc-utils.c                           |    2 +-
 10 files changed, 11 insertions(+), 14 deletions(-)
---
diff --git a/libgda/binreloc/gda-binreloc.c b/libgda/binreloc/gda-binreloc.c
index eb49460..21d2f54 100644
--- a/libgda/binreloc/gda-binreloc.c
+++ b/libgda/binreloc/gda-binreloc.c
@@ -229,8 +229,10 @@ gda_gbr_get_file_path (GdaPrefixDir where, ...)
 #endif
 	}
 
-	if (!prefix || !*prefix)
+	if (!prefix || !*prefix) {
+		g_free (prefix);
 		return NULL;
+	}
        
 	/* filename part */
 	size = 10;
diff --git a/libgda/gda-server-provider-extra.c b/libgda/gda-server-provider-extra.c
index 51f860d..42d8942 100644
--- a/libgda/gda-server-provider-extra.c
+++ b/libgda/gda-server-provider-extra.c
@@ -112,7 +112,7 @@ GdaDataHandler *
 gda_server_provider_get_data_handler_default (GdaServerProvider *provider, GdaConnection *cnc,
 					      GType type, const gchar *dbms_type)
 {
-	GdaDataHandler *dh;
+	GdaDataHandler *dh = NULL;
 	if ((type == G_TYPE_INT64) ||
 	    (type == G_TYPE_UINT64) ||
 	    (type == G_TYPE_DOUBLE) ||
diff --git a/libgda/gda-util.c b/libgda/gda-util.c
index db82179..0d604b8 100644
--- a/libgda/gda-util.c
+++ b/libgda/gda-util.c
@@ -1053,8 +1053,6 @@ gda_compute_dml_statements (GdaConnection *cnc, GdaStatement *select_stmt, gbool
 			continue;
 		g_hash_table_insert (fields_hash, selfield->field_name, GINT_TO_POINTER (1));
 
-		gchar *str;
-		str = gda_sql_identifier_quote (selfield->field_name, cnc, NULL, FALSE, FALSE);
 		if (insert_stmt) {
 			GdaSqlField *field;
 			field = gda_sql_field_new (GDA_SQL_ANY_PART (ist));
@@ -1092,8 +1090,6 @@ gda_compute_dml_statements (GdaConnection *cnc, GdaStatement *select_stmt, gbool
 			expr->param_spec = pspec;
 			ust->expr_list = g_slist_append (ust->expr_list, expr);
 		}
-
-		g_free (str);
 	}
 	g_hash_table_destroy (fields_hash);
 
diff --git a/libgda/sql-parser/parser.y b/libgda/sql-parser/parser.y
index 45de76d..d675ac3 100644
--- a/libgda/sql-parser/parser.y
+++ b/libgda/sql-parser/parser.y
@@ -206,7 +206,7 @@ create_uni_expr (GdaSqlOperatorType op, GdaSqlExpr *expr) {
 
 static GdaSqlStatement *
 compose_multiple_compounds (GdaSqlStatementCompoundType ctype, GdaSqlStatement *left, GdaSqlStatement *right) {
-	GdaSqlStatement *ret;
+	GdaSqlStatement *ret = NULL;
 	GdaSqlStatementCompound *lc = (GdaSqlStatementCompound*) left->contents;
 	if (lc->compound_type == ctype) {
 		GdaSqlStatementCompound *rc = (GdaSqlStatementCompound*) right->contents;
diff --git a/libgda/sqlite/virtual/gda-vprovider-data-model.c b/libgda/sqlite/virtual/gda-vprovider-data-model.c
index 7fc9e03..279abc9 100644
--- a/libgda/sqlite/virtual/gda-vprovider-data-model.c
+++ b/libgda/sqlite/virtual/gda-vprovider-data-model.c
@@ -583,7 +583,7 @@ virtualOpen (sqlite3_vtab *pVTab, sqlite3_vtab_cursor **ppCursor)
 	cursor = g_new0 (VirtualCursor, 1);
 	cursor->iter = gda_data_model_create_iter (vtable->wrapper);
 	cursor->ncols = gda_data_model_get_n_columns (GDA_DATA_MODEL (vtable->td->real_model));
-	*ppCursor = &(cursor->base);
+	*ppCursor = (sqlite3_vtab_cursor*) cursor;
 	return SQLITE_OK;
 }
 
diff --git a/providers/firebird/parser.y b/providers/firebird/parser.y
index 6c4ecb3..5569c79 100644
--- a/providers/firebird/parser.y
+++ b/providers/firebird/parser.y
@@ -206,7 +206,7 @@ create_uni_expr (GdaSqlOperatorType op, GdaSqlExpr *expr) {
 
 static GdaSqlStatement *
 compose_multiple_compounds (GdaSqlStatementCompoundType ctype, GdaSqlStatement *left, GdaSqlStatement *right) {
-	GdaSqlStatement *ret;
+	GdaSqlStatement *ret = NULL;
 	GdaSqlStatementCompound *lc = (GdaSqlStatementCompound*) left->contents;
 	if (lc->compound_type == ctype) {
 		GdaSqlStatementCompound *rc = (GdaSqlStatementCompound*) right->contents;
diff --git a/providers/skel-implementation/capi/parser.y b/providers/skel-implementation/capi/parser.y
index 4444ade..744022c 100644
--- a/providers/skel-implementation/capi/parser.y
+++ b/providers/skel-implementation/capi/parser.y
@@ -206,7 +206,7 @@ create_uni_expr (GdaSqlOperatorType op, GdaSqlExpr *expr) {
 
 static GdaSqlStatement *
 compose_multiple_compounds (GdaSqlStatementCompoundType ctype, GdaSqlStatement *left, GdaSqlStatement *right) {
-	GdaSqlStatement *ret;
+	GdaSqlStatement *ret = NULL;
 	GdaSqlStatementCompound *lc = (GdaSqlStatementCompound*) left->contents;
 	if (lc->compound_type == ctype) {
 		GdaSqlStatementCompound *rc = (GdaSqlStatementCompound*) right->contents;
diff --git a/samples/XSLT/transform.c b/samples/XSLT/transform.c
index 9ef7d75..49bf1e0 100644
--- a/samples/XSLT/transform.c
+++ b/samples/XSLT/transform.c
@@ -96,7 +96,7 @@ main (int argc, char *argv[])
 static int
 sqlxslt_process_xslt_file_ext (GdaXsltExCont *sql_ctx, const char *inputFile, 
 			       const char *xslFileName, const char *outputFileName) {
-	int ret;
+	int ret = 0;
 	xmlDocPtr doc,res;
 	xsltStylesheetPtr xsltdoc;
 	xsltTransformContextPtr ctxt;
diff --git a/tests/meta-store/common.c b/tests/meta-store/common.c
index dd8c14f..9afb9f4 100644
--- a/tests/meta-store/common.c
+++ b/tests/meta-store/common.c
@@ -108,8 +108,7 @@ meta_changed_cb (GdaMetaStore *store, GSList *changes, gpointer data)
 	}
 	if (expected_changes) {
 		/* expected more changes */
-		el = expected_changes;
-		gchar *estr = (gchar *) el->data;
+		gchar *estr = (gchar *) expected_changes->data;
 		g_print ("Received no change but EXPECTED GdaMetaStoreChange: %s", estr);
 		exit (EXIT_FAILURE);
 	}
diff --git a/tests/test-cnc-utils.c b/tests/test-cnc-utils.c
index 3f60fbf..63225d8 100644
--- a/tests/test-cnc-utils.c
+++ b/tests/test-cnc-utils.c
@@ -328,7 +328,7 @@ test_cnc_setup_db_contents (GdaConnection *cnc, const gchar *data_file, GError *
 gboolean
 test_cnc_load_data_from_file (GdaConnection *cnc, const gchar *table, const gchar *full_file, GError **error)
 {
-	GdaStatement *stmt;
+	GdaStatement *stmt = NULL;
 	GdaSet *params = NULL;
 	GdaDataModel *import;
 	gint nrows, ncols, i;