cron2 has submitted this change. ( 
http://gerrit.openvpn.net/c/openvpn/+/1842?usp=email )

Change subject: Fix extracting IV_MTU
......................................................................

Fix extracting IV_MTU

Commit 06ced3fd accidentally broke extracting the MTU from the peer
info while refactoring the code. Before 06ced3fd the code used
iv_mtu as temporary string and extracted the value into client_max_mtu.
The new code never set client_max_mtu and used iv_mtu as integer instead.

Fix this by setting client_max_mtu again with the real MTU value from
the peer.

Change-Id: Ia2e238a41526ebc178c53d3807103d8b9da475a5
Signed-off-by: Arne Schwabe <[email protected]>
Acked-by: Gert Doering <[email protected]>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1842
Message-Id: <[email protected]>
URL: 
https://www.mail-archive.com/[email protected]/msg38291.html
Signed-off-by: Gert Doering <[email protected]>
---
M src/openvpn/push.c
1 file changed, 2 insertions(+), 3 deletions(-)




diff --git a/src/openvpn/push.c b/src/openvpn/push.c
index 97f352e..5b15f71 100644
--- a/src/openvpn/push.c
+++ b/src/openvpn/push.c
@@ -717,10 +717,9 @@
     }

     /* Push our mtu to the peer if it supports pushable MTUs */
-    int client_max_mtu = 0;
-    unsigned int iv_mtu = peer_info_extract_uint(tls_multi->peer_info, 
"IV_MTU=");
+    int client_max_mtu = peer_info_extract_uint(tls_multi->peer_info, 
"IV_MTU=");

-    if (iv_mtu != 0)
+    if (client_max_mtu != 0)
     {
         push_option_fmt(gc, push_list, M_USAGE, "tun-mtu %d", o->ce.tun_mtu);
         if (client_max_mtu < o->ce.tun_mtu)

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

Gerrit-MessageType: merged
Gerrit-Project: openvpn
Gerrit-Branch: master
Gerrit-Change-Id: Ia2e238a41526ebc178c53d3807103d8b9da475a5
Gerrit-Change-Number: 1842
Gerrit-PatchSet: 2
Gerrit-Owner: plaisthos <[email protected]>
Gerrit-Reviewer: cron2 <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to