GitList

Browse code

Fixes a bug in management_callback_send_cc_message, should be strlen instead of sizeof

Signed-off-by: Eric Thorpe <eric@sparklabs.com>
Acked-by: Arne Schwabe <arne@rfc2549.org>
Message-Id: <20200820014258.38377-1-eric@sparklabs.com>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg20783.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>
(cherry picked from commit 62560e2a29144ffefec231ae7147eb4e9c7dfebf)

Eric Thorpe authored on 2020/08/20 10:42:58
Showing 1 changed files

src/openvpn/init.c index dfa045b..a785934 100644

src/openvpn/init.c

History View file @ 6e1fcca

@@ -312,7 +312,7 @@ management_callback_send_cc_message(void *arg,
                                     const char *parameters)
 {
     struct context *c = (struct context *) arg;
-    size_t len = strlen(command) + 1 + sizeof(parameters) + 1;
+    size_t len = strlen(command) + 1 + strlen(parameters) + 1;
     if (len > PUSH_BUNDLE_SIZE)
     {
         return false;

...	...	@@ -312,7 +312,7 @@ management_callback_send_cc_message(void *arg,
312	312	const char *parameters)
313	313	{
314	314	struct context c = (struct context ) arg;
315		- size_t len = strlen(command) + 1 + sizeof(parameters) + 1;
	315	+ size_t len = strlen(command) + 1 + strlen(parameters) + 1;
316	316	if (len > PUSH_BUNDLE_SIZE)
317	317	{
318	318	return false;