Acked-by: Frank Lichtenheld <fr...@lichtenheld.com>
> Arne Schwabe <a...@rfc2549.org> hat am 07.12.2021 18:02 geschrieben:
> diff --git a/src/openvpn/init.c b/src/openvpn/init.c
> index 31ae250bc..4f08e560a 100644
> --- a/src/openvpn/init.c
> +++ b/src/openvpn/init.c
> @@ -3183,12 +3183,17 @@ do_init_frame(struct context *c)
> #endif
>
> #ifdef ENABLE_FRAGMENT
> - if ((c->options.ce.mssfix || c->options.ce.fragment)
> - && TUN_MTU_SIZE(&c->c2.frame_fragment) != ETHERNET_MTU)
> + if (c->options.ce.fragment > 0 && c->options.ce.mssfix >
> c->options.ce.fragment)
> {
> - msg(M_WARN,
> - "WARNING: normally if you use --mssfix and/or --fragment, you
> should also set --tun-mtu %d (currently it is %d)",
> - ETHERNET_MTU, TUN_MTU_SIZE(&c->c2.frame_fragment));
> + msg(M_WARN, "WARNING: if you use --mssfix and --fragment, you should
> "
> + "set --fragment (%d) larger than --mssfix (%d)",
> + c->options.ce.fragment, c->options.ce.mssfix);
> + }
> + if (c->options.ce.fragment > 0 && c->options.ce.mssfix > 0
> + && c->options.ce.fragment_encap != c->options.ce.mssfix_encap)
> + {
> + msg(M_WARN, "WARNING: if you use --mssfix and --fragment, you should
> "
> + "use the mtu for both or none.");
"the mtu parameter"? Or do you actually mean "the mtu" here?
Regards,
--
Frank Lichtenheld
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
