[Openvpn-devel] [XS] Change in openvpn[master]: Correctly calculate packet id size when epoch is in use
"plaisthos \(Code Review\) via Openvpn-devel" <[email protected]> Tue, 28 Jul 2026 18:47:23 +0000
| Newsgroups | net.sourceforge.lists.openvpn-devel |
|---|---|
| Message-ID | <[email protected]> |
plaisthos has uploaded this change for review. ( http://gerrit.openvpn.net/c/openvpn/+/1829?usp=email ) Change subject: Correctly calculate packet id size when epoch is in use ...................................................................... Correctly calculate packet id size when epoch is in use Even thought packet_id_long_form has a the same size (8) it is not the same header format (32 bit time + 32 bit IV) as the epoch format (16 bit epoch + 48 IV). Use a simple sizeof(uint64_t) to avoid suggesting that it might be the same. Closes: Github #1074 Change-Id: I5b862eabe032eb4d2229ff5b2f4f6af7406f6f4e Signed-off-by: Arne Schwabe <[email protected]> --- M src/openvpn/mtu.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/29/1829/1 diff --git a/src/openvpn/mtu.c b/src/openvpn/mtu.c index e5db8ab..d5fec80 100644 --- a/src/openvpn/mtu.c +++ b/src/openvpn/mtu.c @@ -35,6 +35,7 @@ #include "crypto.h" #include "memdbg.h" +#include "ssl_common.h" /* allocate a buffer for socket or tun layer */ void @@ -51,6 +52,12 @@ calc_packet_id_size_dc(const struct options *options, const struct key_type *kt) { bool tlsmode = options->tls_server || options->tls_client; + bool epoch = options->imported_protocol_flags & CO_EPOCH_DATA_KEY_FORMAT; + + if (epoch) + { + return sizeof(uint64_t); + } bool packet_id_long_form = !tlsmode || cipher_kt_mode_ofb_cfb(kt->cipher); -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/1829?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: I5b862eabe032eb4d2229ff5b2f4f6af7406f6f4e Gerrit-Change-Number: 1829 Gerrit-PatchSet: 1 Gerrit-Owner: plaisthos <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> _______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel