Attention is currently required from: cron2, flichtenheld, ordex. plaisthos has posted comments on this change. ( http://gerrit.openvpn.net/c/openvpn/+/797?usp=email )
Change subject: Use XOR instead of concatenation for calculation of IV from implicit IV ...................................................................... Patch Set 3: (2 comments) File src/openvpn/crypto.c: http://gerrit.openvpn.net/c/openvpn/+/797/comment/9dbf827d_d3cd9727 : PS3, Line 435: const size_t packet_iv_len = packet_id_size(false); > nit: since you are keeping this declaration here, I think it is more > descriptive to use packet_iv_le […] I don't really follow. Where do you want me to use packet_iv_len and instead of what? File src/openvpn/ssl.c: http://gerrit.openvpn.net/c/openvpn/+/797/comment/24afc1eb_4d268ad7 : PS3, Line 1566: /* The first bytes of the IV are filled with the packet id */ > is the extension to 96 bits implicit IV coming with a future patch? (i.e. > DATA_V3 patchset?) […] Yes. That is being done in a one of the future patches. -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/797?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: I74216d776d3e0a8dc987ec7b1671c8e8dcccdbd6 Gerrit-Change-Number: 797 Gerrit-PatchSet: 3 Gerrit-Owner: plaisthos <[email protected]> Gerrit-Reviewer: flichtenheld <[email protected]> Gerrit-CC: cron2 <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> Gerrit-CC: ordex <[email protected]> Gerrit-Attention: cron2 <[email protected]> Gerrit-Attention: flichtenheld <[email protected]> Gerrit-Attention: ordex <[email protected]> Gerrit-Comment-Date: Fri, 15 Nov 2024 11:59:33 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: ordex <[email protected]> Gerrit-MessageType: comment
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
