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

Change-Id: I15dd103fe7542f70113c1d5e539a99f8cd062be4
BUG: 1163543
Signed-off-by: Joseph Fernandes <josferna@redhat.com>
Reviewed-on: http://review.gluster.org/10870
Tested-by: NetBSD Build System
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>

1 files changed, 12 insertions, 2 deletions

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;