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 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) (limited to 'libglusterfs/src/gfdb/gfdb_data_store.c') 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; -- cgit