ikev1: Minor code optimization in task manager

Signed-off-by: Thomas Egerer <thomas.egerer@secunet.com>

diff --git a/src/libcharon/sa/ikev1/task_manager_v1.c b/src/libcharon/sa/ikev1/task_manager_v1.c
index 3b0c1cfd1b63941f3d4224f7630e0bbdb5fb83f0..ba393196df3b256c1f315a417afab0eae8b1f099 100644 (file)
--- a/src/libcharon/sa/ikev1/task_manager_v1.c
+++ b/src/libcharon/sa/ikev1/task_manager_v1.c
@@ -367,7 +367,7 @@ static status_t retransmit_packet(private_task_manager_t *this, uint32_t seqnr,
        send_packets(this, packets);
        lib->scheduler->schedule_job_ms(lib->scheduler, (job_t*)
                        retransmit_job_create(seqnr, this->ike_sa->get_id(this->ike_sa)), t);
-       return NEED_MORE;
+       return SUCCESS;
 }
 
 METHOD(task_manager_t, retransmit, status_t,
@@ -380,10 +380,9 @@ METHOD(task_manager_t, retransmit, status_t,
        {
                status = retransmit_packet(this, seqnr, this->initiating.mid,
                                        this->initiating.retransmitted, this->initiating.packets);
-               if (status == NEED_MORE)
+               if (status == SUCCESS)
                {
                        this->initiating.retransmitted++;
-                       status = SUCCESS;
                }
        }
        if (seqnr == this->responding.seqnr &&
@@ -391,10 +390,9 @@ METHOD(task_manager_t, retransmit, status_t,
        {
                status = retransmit_packet(this, seqnr, this->responding.mid,
                                        this->responding.retransmitted, this->responding.packets);
-               if (status == NEED_MORE)
+               if (status == SUCCESS)
                {
                        this->responding.retransmitted++;
-                       status = SUCCESS;
                }
        }
        return status;
@@ -685,13 +683,9 @@ METHOD(task_manager_t, initiate, status_t,
                message->destroy(message);
                return retransmit(this, this->initiating.seqnr);
        }
-       if (keep)
-       {       /* keep the packet for retransmission, the responder might request it */
-               send_packets(this, this->initiating.packets);
-       }
-       else
+       send_packets(this, this->initiating.packets);
+       if (!keep)
        {
-               send_packets(this, this->initiating.packets);
                clear_packets(this->initiating.packets);
        }
        message->destroy(message);