In process_v2_CREATE_CHILD_SA_request_continue_2, if ike is NULL: - the comment I added says that the result is failure
- but the result returned is STF_OK (most analogous cases return STF_INTERNAL_ERROR) One of these is wrong and should be fixed. A comment doesn't affect correctness (the part I added). Is the STF_OK correct? If so, we need to replace the incorrect comment with a better description of what's up. If the STF_OK is incorrect, what should the code do? If the case is actually impossible, I suggest a passert. That would simplify the code. _______________________________________________ Swan-dev mailing list Swan-dev@lists.libreswan.org https://lists.libreswan.org/mailman/listinfo/swan-dev