Attention is currently required from: flichtenheld, ordex, plaisthos. cron2 has posted comments on this change. ( http://gerrit.openvpn.net/c/openvpn/+/1027?usp=email )
Change subject: dco_linux: avoid bogus text when netlink message is not parsed ...................................................................... Patch Set 2: (1 comment) File src/openvpn/forward.c: http://gerrit.openvpn.net/c/openvpn/+/1027/comment/641ebff7_a3997926 : PS2, Line 1254: /* no message was parsed - bail out */ I would change this to ``` /* no message for us to handle - platform specific code has logged details */ ``` because we do not really "bail out", we just do nothing because nothing is to be done. -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/1027?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: I568faa12a5960e8b69de23c2df413b70b231592c Gerrit-Change-Number: 1027 Gerrit-PatchSet: 2 Gerrit-Owner: ordex <[email protected]> Gerrit-Reviewer: cron2 <[email protected]> Gerrit-Reviewer: flichtenheld <[email protected]> Gerrit-Reviewer: plaisthos <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> Gerrit-Attention: plaisthos <[email protected]> Gerrit-Attention: flichtenheld <[email protected]> Gerrit-Attention: ordex <[email protected]> Gerrit-Comment-Date: Fri, 23 May 2025 06:09:55 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
