recoverd: Print banning message only after verifying pnn

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
(cherry picked from commit 4be8dff3a4451192f838497b4747273685959bed)

diff --git a/server/ctdb_recoverd.c b/server/ctdb_recoverd.c
index c660aac271225b4b4170b8c0facd30c292d6c5e9..7e5a98fe93865daa04fe01ed47cf5d95e4920393 100644 (file)
--- a/server/ctdb_recoverd.c
+++ b/server/ctdb_recoverd.c
@@ -80,13 +80,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;