[Openvpn-devel] [L] Change in openvpn[master]: Add lookup of multi session by session id

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