This is one of the "integer" patches that uncovered an actual logic
error - checking the wrong variable for the trigger "can I do epoch data?".
This did not hurt so far (because the other checks ensure the right
outcome) but I think it de-fuses the "data_v2 must be enabled!" sanity
check that would lead to not detecting a non-compliant client. So it's
good we found this in time :-)
Not tested beyond "looks reasonable, the BBs say it compiles fine"
(the netbsd test fails are due to stuck t_client process from a previous
test run abort).
Your patch has been applied to the master branch.
commit c124e50317e530394d9011ddde5653ccfdd94482
Author: Frank Lichtenheld
Date: Thu Oct 16 12:33:03 2025 +0200
init: Fix datav2_enabled check in options import
Signed-off-by: Frank Lichtenheld <[email protected]>
Acked-by: Arne Schwabe <[email protected]>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1273
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg33365.html
Signed-off-by: Gert Doering <[email protected]>
--
kind regards,
Gert Doering
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
