[Openvpn-devel] [PATCH 3/3 v2] doc/options: clean up documentation for --proto and related options

2022-02-10 Thread Frank Lichtenheld
The family specific options were generally omitted. Signed-off-by: Frank Lichtenheld --- doc/man-sections/client-options.rst | 5 + doc/man-sections/link-options.rst | 5 - src/openvpn/options.c | 17 + 3 files changed, 18 insertions(+), 9 deletions(-)

[Openvpn-devel] [PATCH 2.5] doc/Makefile: rebuild rst docs if input files change

2022-02-10 Thread Frank Lichtenheld
For now the dependencies are statically defined, which should be fine and is still a much better solution than to have no dependencies. Also delete empty example-fingerprint.rst. (cherry picked from commit 529cc6a52fc45a1a67abb5c91819ba4fb2f631e3) Signed-off-by: Frank Lichtenheld --- doc/Makefi

[Openvpn-devel] [PATCH v4 3/8] Add mtu paramter to --fragment and change fragment calculation

2022-02-10 Thread Arne Schwabe
Instead relying on the link_mtu_dynamic field and its calculation in the frame struct, add a new field max_fragment_size and add a calculation of it similar to mssfix. Also whenever mssfix value is calculated, we also want to calculate the values for fragment as both options need to be calculated

[Openvpn-devel] [PATCH v4 7/8] Remove frame->link_mtu

2022-02-10 Thread Arne Schwabe
Signed-off-by: Arne Schwabe --- src/openvpn/comp.c| 8 src/openvpn/comp.h| 2 -- src/openvpn/forward.c | 4 ++-- src/openvpn/init.c| 39 +++ src/openvpn/mtu.c | 26 -- src/openvpn/mtu.h | 22 --

[Openvpn-devel] [PATCH v4 5/8] Use new frame header methods to calculate OCC_MTU_LOAD payload size

2022-02-10 Thread Arne Schwabe
Signed-off-by: Arne Schwabe --- src/openvpn/occ.c | 31 +++ 1 file changed, 23 insertions(+), 8 deletions(-) diff --git a/src/openvpn/occ.c b/src/openvpn/occ.c index 6fc5e003..b7670356 100644 --- a/src/openvpn/occ.c +++ b/src/openvpn/occ.c @@ -199,8 +199,11 @@ check_s

[Openvpn-devel] [PATCH v4 6/8] Remove extra_link from frame

2022-02-10 Thread Arne Schwabe
The previous commits removed any reads from this variable. So we can now safely remove it. Signed-off-by: Arne Schwabe --- src/openvpn/init.c | 19 --- src/openvpn/mtu.c | 1 - src/openvpn/mtu.h | 13 - src/openvpn/socks.c | 11 +-- src/openvpn/socks.h |

[Openvpn-devel] [PATCH v4 1/8] Replace TUN_MTU_SIZE with frame->tun_mtu

2022-02-10 Thread Arne Schwabe
This always uses the configured MTU size instead relying on the calculated MTU size. Patch v4: Fix a few overlooked TUN_MTU_SIZE. Signed-off-by: Arne Schwabe --- src/openvpn/forward.c | 2 +- src/openvpn/init.c| 20 ++-- src/openvpn/mtu.c | 4 ++-- src/openvpn/mtu.h

[Openvpn-devel] [PATCH v4 4/8] Update fragment and mssfix related warnings

2022-02-10 Thread Arne Schwabe
The warning that fragment/mssfix needs also tun-mtu set to 1500 makes little sense. Remove it completely. Instead warn if there are incosistencies between --fragment and mssfix. Patch v2: clarify the mssfix and fragment mtu warning message Patch v4: Rebase Signed-off-by: Arne Schwabe --- src/op

[Openvpn-devel] [PATCH v4 2/8] Change the default for mssfix to mssfix 1492 mtu

2022-02-10 Thread Arne Schwabe
The current default is 1450, which translates to 1478 byte packets for udp4 and 1498 byte packets for udp6. This commit changes the mssfix default to take the outer IP overhead into account as well and changes the target to 1492. 1492 was picked in our community meeting for being a very common enca

[Openvpn-devel] [PATCH v4 8/8] Remove frame.extra_frame and frame.extra_buffer

2022-02-10 Thread Arne Schwabe
Signed-off-by: Arne Schwabe --- src/openvpn/comp.c | 7 -- src/openvpn/comp.h | 2 -- src/openvpn/crypto.c| 37 --- src/openvpn/fragment.c | 3 --- src/openvpn/init.c | 56 - src/openvpn/mtu.c | 14 --