diff options
author | Lev Stipakov <lstipakov@gmail.com> | 2015-03-05 22:33:56 +0200 |
---|---|---|
committer | Gert Doering <gert@greenie.muc.de> | 2015-03-07 12:33:49 +0100 |
commit | 785297bf3c8a93ad2fca2aeae574082dc97183e6 (patch) | |
tree | 6075d893ece33606ff943f08f67d84ee6930a77b /src/openvpn/options.c | |
parent | d7d61b4c2d1f1dd47e779ca38b936f9b99042c82 (diff) | |
download | openvpn-785297bf3c8a93ad2fca2aeae574082dc97183e6.tar.gz openvpn-785297bf3c8a93ad2fca2aeae574082dc97183e6.tar.xz openvpn-785297bf3c8a93ad2fca2aeae574082dc97183e6.zip |
Fix mssfix default value in connection_list context
Due to this bug, mssfix hasn't been assigned to fragment value
and used default value (1450) instead. As a consequence, TCP packets
get fragmented, which causes performance penalty.
Since dual stack patch (commit 23d61c56b9fd218c39ad151b01b7e2d6690e6093)
OpenVPN uses options->connection_list, even for single remote.
This fix assigns mssfix value to fragment value for connection_entry
inside connection_list instead of connection_entry inside options struct
(which does not work for connection_list case).
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <1425587636-23338-1-git-send-email-lstipakov@gmail.com>
URL: http://article.gmane.org/gmane.network.openvpn.devel/9507
Signed-off-by: Gert Doering <gert@greenie.muc.de>
(cherry picked from commit d384a9587951617d12e31e0a18050bd86402d5df)
Diffstat (limited to 'src/openvpn/options.c')
-rw-r--r-- | src/openvpn/options.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/openvpn/options.c b/src/openvpn/options.c index dd79da4..ec84241 100644 --- a/src/openvpn/options.c +++ b/src/openvpn/options.c @@ -2400,7 +2400,7 @@ options_postprocess_mutate_ce (struct options *o, struct connection_entry *ce) { #ifdef ENABLE_FRAGMENT if (ce->fragment) - o->ce.mssfix = ce->fragment; + ce->mssfix = ce->fragment; #else msg (M_USAGE, "--mssfix must specify a parameter"); #endif |