Hi,
On 17/08/2022 15:53, Arne Schwabe wrote:
Commit 8c72d7981 changed the push_peer_info_detail to have an
additional level for P2P NCP and shifting most of the other levels
with 1. The check for UV_ and IV_PLAT_VER was not changed accordingly.
Fixes: 8c72d7981 ("Support NCP in pure P2P VPN setups")
Signed-off-by: Arne Schwabe <a...@rfc2549.org>
Same as v1, just rebased and with an extended commit message.
Tested by setting some UV_VAR and seeing it being sent only with
--push-peer-info.
Acked-by: Antonio Quartulli <a...@unstable.cc>
---
src/openvpn/ssl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/openvpn/ssl.c b/src/openvpn/ssl.c
index 77f572a40..33e145b3f 100644
--- a/src/openvpn/ssl.c
+++ b/src/openvpn/ssl.c
@@ -2036,7 +2036,7 @@ push_peer_info(struct buffer *buf, struct tls_session
*session)
{
if ((((strncmp(e->string, "UV_", 3) == 0
|| strncmp(e->string, "IV_PLAT_VER=",
sizeof("IV_PLAT_VER=") - 1) == 0)
- && session->opt->push_peer_info_detail >= 2)
+ && session->opt->push_peer_info_detail > 2)
|| (strncmp(e->string, "IV_GUI_VER=",
sizeof("IV_GUI_VER=") - 1) == 0)
|| (strncmp(e->string, "IV_SSO=", sizeof("IV_SSO=")
- 1) == 0)
)
--
Antonio Quartulli
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
