diff options
| author | Lev Stipakov <lstipakov@gmail.com> | 2014-12-08 18:48:45 +0200 |
|---|---|---|
| committer | David Sommerseth <davids@redhat.com> | 2014-12-08 22:18:20 +0100 |
| commit | 09cf2ec5c09d35c72f2af0d988de8152378a182a (patch) | |
| tree | 03f46da3e6d4ede396b428e28c20f005dcdf3ab6 /src/openvpn/multi.h | |
| parent | 1b9541922ad6ff6ee46c84f43cd23b7064f7919d (diff) | |
| download | openvpn-09cf2ec5c09d35c72f2af0d988de8152378a182a.tar.gz openvpn-09cf2ec5c09d35c72f2af0d988de8152378a182a.tar.xz openvpn-09cf2ec5c09d35c72f2af0d988de8152378a182a.zip | |
Prevent memory drain for long lasting floating sessions
For every float event we generate prefix, which allocates 256 + 64
bytes. That memory is reclaimed when client disconnects, so long lasting
and constantly floating sessions drain memory.
As a fix use preallocated buffer inside multi_instance for storing
multi_prefix.
Signed-off-by: Lev Stipakov <lstipakov@gmail.com>
Acked-by: Steffan Karger <steffan.karger@fox-it.com>
Message-Id: 1418057325-13265-1-git-send-email-lstipakov@gmail.com
URL: http://article.gmane.org/gmane.network.openvpn.devel/9321
Signed-off-by: David Sommerseth <davids@redhat.com>
Diffstat (limited to 'src/openvpn/multi.h')
| -rw-r--r-- | src/openvpn/multi.h | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/openvpn/multi.h b/src/openvpn/multi.h index ad7f700..32b89d2 100644 --- a/src/openvpn/multi.h +++ b/src/openvpn/multi.h @@ -42,6 +42,8 @@ #include "mtcp.h" #include "perf.h" +#define MULTI_PREFIX_MAX_LENGTH 256 + /* * Walk (don't run) through the routing table, * deleting old entries, and possibly multi_instance @@ -80,7 +82,7 @@ struct multi_instance { struct mroute_addr real; /**< External network address of the * remote peer. */ ifconfig_pool_handle vaddr_handle; - const char *msg_prefix; + char msg_prefix[MULTI_PREFIX_MAX_LENGTH]; /* queued outgoing data in Server/TCP mode */ unsigned int tcp_rwflags; @@ -445,10 +447,10 @@ static inline void set_prefix (struct multi_instance *mi) { #ifdef MULTI_DEBUG_EVENT_LOOP - if (mi->msg_prefix) + if (mi->msg_prefix[0]) printf ("[%s]\n", mi->msg_prefix); #endif - msg_set_prefix (mi->msg_prefix); + msg_set_prefix (mi->msg_prefix[0] ? mi->msg_prefix : NULL); } static inline void |