<div dir="ltr"><div><div><div>My bad.</div><div><br></div>Can you try the attached which I'll call fix #1.  It eliminates a continue that would cause code setting STTTTTTTTTTTTTTTTT=NULL to be skipped (as you can tell by the duplicate T's I'm running F26 with a kvm in the background :-).<br></div></div>Fix #2, is to be robust against continue; and fix #3 is to be rid of code relying on this macro quirk.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 19 October 2017 at 08:16, Tuomo Soini <span dir="ltr"><<a href="mailto:tis@foobar.fi" target="_blank">tis@foobar.fi</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
git bisect start<br>
# bad: [<wbr>2baaa653a9961acda26f8b38e780db<wbr>4acbab774a] testing: cleanup<br>
xauth-pluto-26 to avoid race on ipsec stop git bisect bad<br>
2baaa653a9961acda26f8b38e780db<wbr>4acbab774a # good:<br>
[<wbr>961c67d31dcbbef9846df63a136744<wbr>09acfc6c2c] testing: for<br>
kvm-install-hive build into $(KVM_OBJDIR) git bisect good<br>
961c67d31dcbbef9846df63a136744<wbr>09acfc6c2c # bad:<br>
[<wbr>1ce066c9f60c7a50b3f8f39870eb7e<wbr>ead19d2c2d] Merge branch 'master' of<br>
vault.libreswan.org:/srv/src/<wbr>libreswan git bisect bad<br>
1ce066c9f60c7a50b3f8f39870eb7e<wbr>ead19d2c2d # bad:<br>
[<wbr>23ac09a67dea13c7ce12e67f62ee8d<wbr>b5960e6244] state: change struct<br>
state_entry into a true double linked list git bisect bad<br>
23ac09a67dea13c7ce12e67f62ee8d<wbr>b5960e6244 # good:<br>
[<wbr>38f8c2f811761583ec5b5398f22bbf<wbr>f9964fb46a] state: when re-hashing a<br>
state, only re-hash the cookies git bisect good<br>
38f8c2f811761583ec5b5398f22bbf<wbr>f9964fb46a # first bad commit:<br>
[<wbr>23ac09a67dea13c7ce12e67f62ee8d<wbr>b5960e6244] state: change struct<br>
state_entry into a true double linked list<br>
<br>
This hits on ikev1.c line 1515:<br>
<br>
passert(STATE_IKE_FLOOR <= from_state && from_state <=<br>
STATE_IKEv1_ROOF);<br>
<br>
But even with last stable version I can't get that ikev1 connection<br>
working at all, logged problem is:<br>
<br>
#345: byte 2 of ISAKMP Hash Payload should have been zero, but was not<br>
(ignored)<br>
#345: length of ISAKMP Hash Payload is larger than can fit<br>
<br>
This exact tunnel hasn't been working after 3.21 release, so it was<br>
broken between 3.21 and 5290022.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Tuomo Soini <<a href="mailto:tis@foobar.fi">tis@foobar.fi</a>><br>
Foobar Linux services<br>
<a href="tel:%2B358%2040%205240030" value="+358405240030">+358 40 5240030</a><br>
Foobar Oy <<a href="https://foobar.fi/" rel="noreferrer" target="_blank">https://foobar.fi/</a>><br>
______________________________<wbr>_________________<br>
Swan-dev mailing list<br>
<a href="mailto:Swan-dev@lists.libreswan.org">Swan-dev@lists.libreswan.org</a><br>
<a href="https://lists.libreswan.org/mailman/listinfo/swan-dev" rel="noreferrer" target="_blank">https://lists.libreswan.org/<wbr>mailman/listinfo/swan-dev</a><br>
</font></span></blockquote></div><br></div>