child-rekey: Remove redundant migrate() call for child-create sub-task

When retrying due to a DH group mismatch this is already done by the
child-create task itself.  And in other cases where the task returns
NEED_MORE we actually will need access to a possible proposal to properly
delete it.

diff --git a/src/libcharon/sa/ikev2/tasks/child_rekey.c b/src/libcharon/sa/ikev2/tasks/child_rekey.c
index c806e19ca3fb912d2c5072ae1687410bc331fea5..4dc9aaf33ecabe4ff348bf81b9ced288ab5ceb0d 100644 (file)
--- a/src/libcharon/sa/ikev2/tasks/child_rekey.c
+++ b/src/libcharon/sa/ikev2/tasks/child_rekey.c
@@ -334,8 +334,7 @@ METHOD(task_t, process_i, status_t,
        if (this->child_create->task.process(&this->child_create->task,
                                                                                 message) == NEED_MORE)
        {
-               /* bad DH group while rekeying, try again */
-               this->child_create->task.migrate(&this->child_create->task, this->ike_sa);
+               /* bad DH group while rekeying, retry, or failure requiring deletion */
                return NEED_MORE;
        }
        if (message->get_payload(message, PLV2_SECURITY_ASSOCIATION) == NULL)