Attention is currently required from: plaisthos. flichtenheld has posted comments on this change by plaisthos. ( http://gerrit.openvpn.net/c/openvpn/+/1727?usp=email )
Change subject: Add lookup of multi session by session id ...................................................................... Patch Set 26: Code-Review-1 (4 comments) File src/openvpn/multi.h: http://gerrit.openvpn.net/c/openvpn/+/1727/comment/94ecc620_97e9f078?usp=email : PS26, Line 138: /** If this is multi_instance is hashed in the sid lookup table the session Remove the first "is" and the second "session id" File src/openvpn/multi.c: http://gerrit.openvpn.net/c/openvpn/+/1727/comment/d89279c8_857f730e?usp=email : PS26, Line 431: && dco_enabled(&mi->context.options)) is this an intended change or a debugging left-over? File src/openvpn/sid_hash.h: http://gerrit.openvpn.net/c/openvpn/+/1727/comment/48c942e9_b37ff5c7?usp=email : PS26, Line 23: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. old header http://gerrit.openvpn.net/c/openvpn/+/1727/comment/9082db0b_a7b8f735?usp=email : PS26, Line 61: struct hash_element *he_sid = hash_lookup_fast(m->sid_hash, sid_bucket, sid, sid_hv); should use `multi_hash_sid_lookup()` -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/1727?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: Idb59ecd119331b198792ad1379bec8600211651b Gerrit-Change-Number: 1727 Gerrit-PatchSet: 26 Gerrit-Owner: plaisthos <[email protected]> Gerrit-Reviewer: flichtenheld <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> Gerrit-Attention: plaisthos <[email protected]> Gerrit-Comment-Date: Fri, 07 Aug 2026 15:57:10 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
