recoverd: Print banning message only after verifying pnn
authorAmitay Isaacs <amitay@gmail.com>
Mon, 24 Jun 2013 04:18:58 +0000 (14:18 +1000)
committerAmitay Isaacs <amitay@gmail.com>
Tue, 2 Jul 2013 07:41:54 +0000 (17:41 +1000)
Signed-off-by: Amitay Isaacs <amitay@gmail.com>
(cherry picked from commit 4be8dff3a4451192f838497b4747273685959bed)

server/ctdb_recoverd.c

index 525377b9a6f882891e5b3de3765f6742ad90bb53..6487b8298b3c233971c27dc0a8d8c26820585551 100644 (file)
@@ -82,13 +82,13 @@ static void ctdb_ban_node(struct ctdb_recoverd *rec, uint32_t pnn, uint32_t ban_
        struct ctdb_context *ctdb = rec->ctdb;
        struct ctdb_ban_time bantime;
        
-       DEBUG(DEBUG_NOTICE,("Banning node %u for %u seconds\n", pnn, ban_time));
-
        if (!ctdb_validate_pnn(ctdb, pnn)) {
                DEBUG(DEBUG_ERR,("Bad pnn %u in ctdb_ban_node\n", pnn));
                return;
        }
 
+       DEBUG(DEBUG_NOTICE,("Banning node %u for %u seconds\n", pnn, ban_time));
+
        bantime.pnn  = pnn;
        bantime.time = ban_time;