<div dir="ltr"><div>Hi all,<br></div><div dir="ltr"><br></div><div dir="ltr">сре, 6. феб 2019. у 23:21 D. Hugh Redelmeier <<a href="mailto:hugh@mimosa.com">hugh@mimosa.com</a>> је написао/ла:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">struct state has:<br>
<br>
        bool st_peer_wants_null;                /* We received IDr payload of type ID_NULL (and we allow POLICY_AUTH_NULL */<br>
<br>
This was added in commit 96015a9f0753d21938960b71e0b81f071833d8ed from<br>
almost a year ago.<br>
<br>
This field is tested but never set.<br>
</blockquote><div><br></div><div><div>It is set in ikev2.c, function</div><div>static bool decode_peer_id_counted(struct ike_sa *ike,<br>                   struct msg_digest *md, int depth) ?</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">Of course it is initialized to false when a struct state is created.<br>
<br>
It was set in 96015a9f0753d21938960b71e0b81f071833d8ed by<br>
ikev2_decode_peer_id_and_certs(), something that no longer exists.<br>
<br>
Is this a bug in our current code?<br>
<br>
- Should we delete code that only executes when st_peer_wants_null is true?<br>
<br>
- Should we add code that sets st_peer_wants_null to true?<br>
<br>
- Is st_peer_wants_null needed for some feature to be added in the future?<br>
<br>
Have I missed something?<br>
_______________________________________________<br>
Swan-dev mailing list<br>
<a href="mailto:Swan-dev@lists.libreswan.org" target="_blank">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/mailman/listinfo/swan-dev</a><br>
</blockquote></div></div>