From 3b4980919bf23b952fa5155bc8f9d59613cbafb7 Mon Sep 17 00:00:00 2001 From: Joseph Fernandes Date: Thu, 21 May 2015 11:17:59 +0530 Subject: tier/tier.t: Fixing tier.t crash in regression runs 1) If the database file exists a. Dont try re-creating the db schema b. Dont try re-configuring the db. 2) Dont assert in fini_db () when connection is NULL >> Signed-off-by: Joseph Fernandes >> Reviewed-on: http://review.gluster.org/10870 >> Tested-by: NetBSD Build System >> Tested-by: Gluster Build System >> Reviewed-by: Vijay Bellur >> Signed-off-by: Joseph Fernandes >>Change-Id: Idd2833f55caaa6b3a77d935d877d6c4d2994da6a Change-Id: Ia2e044fce4e41bccc8fdada1cb21f240fdbd55df BUG: 1225077 Signed-off-by: Joseph Fernandes Reviewed-on: http://review.gluster.org/10992 Reviewed-by: Vijay Bellur Tested-by: Vijay Bellur --- libglusterfs/src/gfdb/gfdb_data_store.c | 14 ++++++++++++-- libglusterfs/src/gfdb/gfdb_sqlite3.c | 11 +++++------ 2 files changed, 17 insertions(+), 8 deletions(-) (limited to 'libglusterfs') diff --git a/libglusterfs/src/gfdb/gfdb_data_store.c b/libglusterfs/src/gfdb/gfdb_data_store.c index ec92f7a4d20..5699b750068 100644 --- a/libglusterfs/src/gfdb/gfdb_data_store.c +++ b/libglusterfs/src/gfdb/gfdb_data_store.c @@ -55,6 +55,16 @@ do {\ GF_ASSERT (_conn_node->gfdb_connection.gf_db_connection);\ } while (0) +/* Checks the sanity of the connection node and goto */ +#define CHECK_CONN_NODE_GOTO(_conn_node, label)\ +do {\ + if (!_conn_node) {\ + goto label;\ + };\ + if (!_conn_node->gfdb_connection.gf_db_connection) {\ + goto label;\ + };\ +} while (0) /*Check if the conn node is first in the list*/ #define IS_FIRST_NODE(db_conn_list, _conn_node)\ @@ -297,7 +307,7 @@ fini_db (gfdb_conn_node_t *_conn_node) int ret = -1; gfdb_db_operations_t *db_operations_t = NULL; - CHECK_CONN_NODE(_conn_node); + CHECK_CONN_NODE_GOTO (_conn_node, empty); db_operations_t = &_conn_node->gfdb_connection.gfdb_db_operations; @@ -316,7 +326,7 @@ fini_db (gfdb_conn_node_t *_conn_node) gf_log (GFDB_DATA_STORE, GF_LOG_ERROR, "Failed deleting connection node from list"); } - +empty: ret = 0; out: return ret; diff --git a/libglusterfs/src/gfdb/gfdb_sqlite3.c b/libglusterfs/src/gfdb/gfdb_sqlite3.c index b9de3d3a3a6..02d60857498 100644 --- a/libglusterfs/src/gfdb/gfdb_sqlite3.c +++ b/libglusterfs/src/gfdb/gfdb_sqlite3.c @@ -436,9 +436,9 @@ gf_sqlite3_init (dict_t *args, void **db_conn) { } /* If the file exist we skip the config part - * and create the schema if NOT Present*/ + * and creation of the schema */ if (is_dbfile_exist) - goto create_table; + goto db_exists; /*Apply sqlite3 params to database*/ @@ -450,17 +450,16 @@ gf_sqlite3_init (dict_t *args, void **db_conn) { goto out; } -/*Create the schema if NOT present*/ -create_table: - + /*Create the schema if NOT present*/ ret = create_filetable (sql_conn->sqlite3_db_conn); if (ret) { gf_log (GFDB_STR_SQLITE3, GF_LOG_ERROR, "Failed Creating %s Table", GF_FILE_TABLE); goto out; } - ret = 0; +db_exists: + ret = 0; out: if (ret) { gf_sqlite3_fini ((void **)&sql_conn); -- cgit