Moved set_state after the DBG0 statement, so that the message gets logged also for...

diff --git a/src/charon/sa/tasks/ike_auth.c b/src/charon/sa/tasks/ike_auth.c
index 9f8fc89a21200f9ece3721b5fea34d289598dd88..2c15ac765ebbf5aa208e02cdd9ed41f3a3c5d6db 100644 (file)
--- a/src/charon/sa/tasks/ike_auth.c
+++ b/src/charon/sa/tasks/ike_auth.c
@@ -730,7 +730,6 @@ static status_t build_r(private_ike_auth_t *this, message_t *message)
                                                                chunk_empty);
                        return FAILED;
                }
-               this->ike_sa->set_state(this->ike_sa, IKE_ESTABLISHED);
                DBG0(DBG_IKE, "IKE_SA %s[%d] established between %H[%Y]...%H[%Y]",
                         this->ike_sa->get_name(this->ike_sa),
                         this->ike_sa->get_unique_id(this->ike_sa),
@@ -738,6 +737,7 @@ static status_t build_r(private_ike_auth_t *this, message_t *message)
                         this->ike_sa->get_my_id(this->ike_sa),
                         this->ike_sa->get_other_host(this->ike_sa),
                         this->ike_sa->get_other_id(this->ike_sa));
+               this->ike_sa->set_state(this->ike_sa, IKE_ESTABLISHED);
                charon->bus->ike_updown(charon->bus, this->ike_sa, TRUE);
                return SUCCESS;
        }
@@ -909,7 +909,6 @@ static status_t process_i(private_ike_auth_t *this, message_t *message)
                        DBG1(DBG_IKE, "final authorization hook forbids IKE_SA, cancelling");
                        return FAILED;
                }
-               this->ike_sa->set_state(this->ike_sa, IKE_ESTABLISHED);
                DBG0(DBG_IKE, "IKE_SA %s[%d] established between %H[%Y]...%H[%Y]",
                         this->ike_sa->get_name(this->ike_sa),
                         this->ike_sa->get_unique_id(this->ike_sa),
@@ -917,6 +916,7 @@ static status_t process_i(private_ike_auth_t *this, message_t *message)
                         this->ike_sa->get_my_id(this->ike_sa),
                         this->ike_sa->get_other_host(this->ike_sa),
                         this->ike_sa->get_other_id(this->ike_sa));
+               this->ike_sa->set_state(this->ike_sa, IKE_ESTABLISHED);
                charon->bus->ike_updown(charon->bus, this->ike_sa, TRUE);
                return SUCCESS;
        }