Attention is currently required from: plaisthos, syzzer.

stipa has posted comments on this change. ( 
http://gerrit.openvpn.net/c/openvpn/+/506?usp=email )

Change subject: Implement support for AEAD tag at the end
......................................................................


Patch Set 6: Code-Review+1

(2 comments)

Patchset:

PS6:
Apart from one comment-related nit-pick, I couldn't find anything wrong. I am 
planning to test this soonish with dco-win + data_v3.


File tests/unit_tests/openvpn/test_ssl.c:

http://gerrit.openvpn.net/c/openvpn/+/506/comment/747528fe_47ac025e :
PS6, Line 268: /* This adds a few more methods than strictly necessary but this 
allows
"that strictly necessary" or "that are not strictly necessary" ?



--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/506?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: I00821d75342daf3f813b829812d648fe298bea81
Gerrit-Change-Number: 506
Gerrit-PatchSet: 6
Gerrit-Owner: plaisthos <[email protected]>
Gerrit-Reviewer: flichtenheld <[email protected]>
Gerrit-Reviewer: stipa <[email protected]>
Gerrit-Reviewer: syzzer <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
Gerrit-Attention: plaisthos <[email protected]>
Gerrit-Attention: syzzer <[email protected]>
Gerrit-Comment-Date: Wed, 31 Jul 2024 11:42:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to