From: Steffan Karger Date: Wed, 8 Jun 2016 12:20:40 +0000 (+0200) Subject: cleanup: remove alloc_buffers argument from multi_top_init() X-Git-Tag: v2.4_alpha1~73 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d92718141287a0f9adcffec5c2fa7d76213162cc;p=thirdparty%2Fopenvpn.git cleanup: remove alloc_buffers argument from multi_top_init() multi_top_init() is always called with alloc_buffers=true, so just remove the argument and alloc unconditionally. Signed-off-by: Steffan Karger Acked-by: Arne Schwabe Acked-by: Gert Doering Message-Id: <1465388443-15484-3-git-send-email-steffan@karger.me> URL: http://article.gmane.org/gmane.network.openvpn.devel/11851 Signed-off-by: Gert Doering --- diff --git a/src/openvpn/mtcp.c b/src/openvpn/mtcp.c index 9926d4761..78e5ccd0a 100644 --- a/src/openvpn/mtcp.c +++ b/src/openvpn/mtcp.c @@ -696,7 +696,7 @@ tunnel_server_tcp (struct context *top) multi_init (&multi, top, true, MC_SINGLE_THREADED); /* initialize our cloned top object */ - multi_top_init (&multi, top, true); + multi_top_init (&multi, top); /* initialize management interface */ init_management_callback_multi (&multi); diff --git a/src/openvpn/mudp.c b/src/openvpn/mudp.c index ce6720604..6e0568e36 100644 --- a/src/openvpn/mudp.c +++ b/src/openvpn/mudp.c @@ -283,7 +283,7 @@ tunnel_server_udp_single_threaded (struct context *top) multi_init (&multi, top, false, MC_SINGLE_THREADED); /* initialize our cloned top object */ - multi_top_init (&multi, top, true); + multi_top_init (&multi, top); /* initialize management interface */ init_management_callback_multi (&multi); diff --git a/src/openvpn/multi.c b/src/openvpn/multi.c index 4c43fcc5b..ba7f2c0a4 100644 --- a/src/openvpn/multi.c +++ b/src/openvpn/multi.c @@ -2859,12 +2859,10 @@ multi_process_per_second_timers_dowork (struct multi_context *m) } void -multi_top_init (struct multi_context *m, const struct context *top, const bool alloc_buffers) +multi_top_init (struct multi_context *m, const struct context *top) { inherit_context_top (&m->top, top); - m->top.c2.buffers = NULL; - if (alloc_buffers) - m->top.c2.buffers = init_context_buffers (&top->c2.frame); + m->top.c2.buffers = init_context_buffers (&top->c2.frame); } void diff --git a/src/openvpn/multi.h b/src/openvpn/multi.h index ec1e7ab02..9d8185d49 100644 --- a/src/openvpn/multi.h +++ b/src/openvpn/multi.h @@ -233,7 +233,7 @@ const char *multi_instance_string (const struct multi_instance *mi, bool null, s void multi_init (struct multi_context *m, struct context *t, bool tcp_mode, int thread_mode); void multi_uninit (struct multi_context *m); -void multi_top_init (struct multi_context *m, const struct context *top, const bool alloc_buffers); +void multi_top_init (struct multi_context *m, const struct context *top); void multi_top_free (struct multi_context *m); struct multi_instance *multi_create_instance (struct multi_context *m, const struct mroute_addr *real);