Attention is currently required from: Max Fillinger.

Steffan Karger has posted comments on this change by Max Fillinger. ( 
https://gerrit.openvpn.net/c/openvpn/+/1985?usp=email )

Change subject: Move check_session_buf_not_used method into the method that 
free the buffer
......................................................................


Patch Set 3: Code-Review+2

(4 comments)

File src/openvpn/ssl.c:

https://gerrit.openvpn.net/c/openvpn/+/1985/comment/4c4d9cbd_ee4d234c?usp=email 
:
PS1, Line 2276:         if (session->opt->pull)
> The uncrustify hook wouldn't let me commit it without this change.
Acknowledged


https://gerrit.openvpn.net/c/openvpn/+/1985/comment/31c52804_38af62f4?usp=email 
:
PS1, Line 2453:                                  || multi->multi_state == 
CAS_SUCCEEDED))
> Uncrustify, again.
Acknowledged


https://gerrit.openvpn.net/c/openvpn/+/1985/comment/f83287b0_82fba0d9?usp=email 
:
PS1, Line 3106:             "still in use (tls_wrap.work.data)");
> Uncrustify, again.
Acknowledged


https://gerrit.openvpn.net/c/openvpn/+/1985/comment/82f0725e_c6627dfe?usp=email 
:
PS1, Line 3250:             "session promoted to %strusted",
> Accidental, I reverted to the old log message. […]
Done



--
To view, visit https://gerrit.openvpn.net/c/openvpn/+/1985?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.openvpn.net/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: openvpn
Gerrit-Branch: release/2.5
Gerrit-Change-Id: I64920ed9f714803604d76c6df6b38cf20ce7626d
Gerrit-Change-Number: 1985
Gerrit-PatchSet: 3
Gerrit-Owner: Max Fillinger <[email protected]>
Gerrit-Reviewer: Steffan Karger <[email protected]>
Gerrit-CC: Arne Schwabe <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
Gerrit-Attention: Max Fillinger <[email protected]>
Gerrit-Comment-Date: Tue, 13 Oct 2026 18:29:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Max Fillinger <[email protected]>
Comment-In-Reply-To: Steffan Karger <[email protected]>
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to