From: Arne Schwabe Date: Mon, 30 May 2022 22:41:49 +0000 (+0200) Subject: Remove useless empty line from CR_RESPONSE message X-Git-Tag: v2.6_beta1~186 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=0dbcaba4f301c21e68a5cd032a4b56eb75c17c37;p=thirdparty%2Fopenvpn.git Remove useless empty line from CR_RESPONSE message The out buffer is not filled and this adds an extra empty line for CR_RESPONSE outputs. Also make minor style fixes. Signed-off-by: Arne Schwabe Acked-by: Gert Doering Message-Id: <20220530224149.201883-1-arne@rfc2549.org> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg24447.html Signed-off-by: Gert Doering --- diff --git a/src/openvpn/manage.c b/src/openvpn/manage.c index 58a17347f..50f162a3b 100644 --- a/src/openvpn/manage.c +++ b/src/openvpn/manage.c @@ -2883,17 +2883,14 @@ management_notify_client_cr_response(unsigned mda_key_id, { gc = gc_new(); - struct buffer out = alloc_buf_gc(256, &gc); msg(M_CLIENT, ">CLIENT:CR_RESPONSE,%lu,%u,%s", mdac->cid, mda_key_id, response); man_output_extra_env(management, "CLIENT"); - if (management->connection.env_filter_level>0) + if (management->connection.env_filter_level > 0) { man_output_peer_info_env(management, mdac); } man_output_env(es, true, management->connection.env_filter_level, "CLIENT"); - management_notify_generic(management, BSTR(&out)); - gc_free(&gc); } } diff --git a/src/openvpn/push.c b/src/openvpn/push.c index 70fd1c3ce..63257348a 100644 --- a/src/openvpn/push.c +++ b/src/openvpn/push.c @@ -224,7 +224,6 @@ receive_cr_response(struct context *c, const struct buffer *buffer) struct env_set *es = session->opt->es; int key_id = get_primary_key(c->c2.tls_multi)->key_id; - management_notify_client_cr_response(key_id, mda, es, m); #endif msg(D_PUSH, "CR response was sent by client ('%s')", m);