<div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Tried it.  It reads well when what is being impaired is a boolean vis:</div><div><br></div><div>       if (IMPAIR(send_bogus_isakmp_flag)) {<br>             hdr.isa_flags |= ISAKMP_FLAGS_RESERVED_BIT6;<br>        }</div><div><br></div><div>however,  think it breaks down for more complex values.  For instance, this would look decidedly weird iv IMPAIR() was used.</div><div><br></div><div> fixup->impair = (impair.v1_hash_exchange == exchange<br>                                               ? impair.v1_hash_payload : SEND_NORMAL);</div><div><br></div><div>So I suspect we're left with impair.send_bogus_isakmp_flag <a href="http://et.al">et.al</a>.</div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 1 Apr 2020 at 19:59, Paul Wouters <<a href="mailto:paul@nohats.ca">paul@nohats.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Works for me<br><br><div dir="ltr">Sent from my iPhone</div><div dir="ltr"><br><blockquote type="cite">On Apr 1, 2020, at 19:38, Andrew Cagney <<a href="mailto:andrew.cagney@gmail.com" target="_blank">andrew.cagney@gmail.com</a>> wrote:<br><br></blockquote></div><blockquote type="cite"><div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 1 Apr 2020 at 18:10, Antony Antony <<a href="mailto:antony@phenome.org" target="_blank">antony@phenome.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Mar 30, 2020 at 12:07:17PM -0400, Andrew Cagney wrote:<br>
> I'm cleaning up the impair code.<br>
> <br>
> Internally, the old style #define names are in upper case vis:<br>
>   #define IMPAIR_REPLAY_FORWARD ...<br>
> and<br>
>   IMPAIR(REPLAY_FORWARD)<br>
> while the new ones (that take parameters) are in lower case vis:<br>
>   extern enum send_impairment impair_ike_key_length_attribute<br>
> I've merged the two but preserved the names.<br>
> <br>
> Should the names be made consistent?<br>
<br>
that would be great.  go for it.<br>
<br></blockquote><div><br></div><div>so how about:</div><div>  IMPAIR(replay_forward)<br></div><div>?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
thanks,<br>
-antony<br>
</blockquote></div></div>
<span>_______________________________________________</span><br><span>Swan-dev mailing list</span><br><span><a href="mailto:Swan-dev@lists.libreswan.org" target="_blank">Swan-dev@lists.libreswan.org</a></span><br><span><a href="https://lists.libreswan.org/mailman/listinfo/swan-dev" target="_blank">https://lists.libreswan.org/mailman/listinfo/swan-dev</a></span><br></div></blockquote></div></blockquote></div>