Fix IKEv1 DELETE subtask creation and processing

diff --git a/src/libcharon/sa/tasks/informational.c b/src/libcharon/sa/tasks/informational.c
index c1a30e4eef98e57c8b20d5f85338540f14e86238..c2c6aa5fa49a613655d1042c99b288c60e27c06b 100644 (file)
--- a/src/libcharon/sa/tasks/informational.c
+++ b/src/libcharon/sa/tasks/informational.c
@@ -94,15 +94,19 @@ METHOD(task_t, process_r, status_t,
                                }
                                continue;
                        case DELETE_V1:
-                               delete = (delete_payload_t*)payload;
-                               if (delete->get_protocol_id(delete) == PROTO_IKE)
+                               if (!this->del)
                                {
-                                       this->del = (task_t*)ike_delete_create(this->ike_sa, FALSE);
-                               }
-                               else
-                               {
-                                       this->del = (task_t*)child_delete_create(this->ike_sa,
-                                                                                                                        PROTO_NONE, 0);
+                                       delete = (delete_payload_t*)payload;
+                                       if (delete->get_protocol_id(delete) == PROTO_IKE)
+                                       {
+                                               this->del = (task_t*)ike_delete_create(this->ike_sa,
+                                                                                                                          FALSE);
+                                       }
+                                       else
+                                       {
+                                               this->del = (task_t*)child_delete_create(this->ike_sa,
+                                                                                                                                PROTO_NONE, 0);
+                                       }
                                }
                                break;
                        default:
@@ -112,7 +116,7 @@ METHOD(task_t, process_r, status_t,
        }
        enumerator->destroy(enumerator);
 
-       if (status == SUCCESS)
+       if (this->del && status == SUCCESS)
        {
                return this->del->process(this->del, message);
        }