[Openvpn-devel] multi.c: use mi->cc_config instead of config variable

Message ID 20200701140517.11176-1-a@unstable.cc
State Accepted
Headers show
Series [Openvpn-devel] multi.c: use mi->cc_config instead of config variable | expand

Commit Message

Antonio Quartulli July 1, 2020, 4:05 a.m. UTC
Commit ("Remove parameter config from multi_client_connect_mda") has
removed the config variable in favour of mi->cc_config, however one
occurence was not changed.

Fix it now by properly using mi->cc_config.

Signed-off-by: Antonio Quartulli <a@unstable.cc>
---
 src/openvpn/multi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Gert Doering July 1, 2020, 4:33 a.m. UTC | #1
Acked-by: Gert Doering <gert@greenie.muc.de>

Test compiled.  Stared at code.  Grumbled.  Ran a small t_server test
("just to be sure").

Your patch has been applied to the master branch.

commit 712f2629c8406c017931a082cd996b439b5dfdc9
Author: Antonio Quartulli
Date:   Wed Jul 1 16:05:17 2020 +0200

     multi.c: use mi->cc_config instead of config variable

     Signed-off-by: Antonio Quartulli <a@unstable.cc>
     Acked-by: Gert Doering <gert@greenie.muc.de>
     Message-Id: <20200701140517.11176-1-a@unstable.cc>
     URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg20180.html
     Signed-off-by: Gert Doering <gert@greenie.muc.de>


--
kind regards,

Gert Doering

Patch

diff --git a/src/openvpn/multi.c b/src/openvpn/multi.c
index 19261c37..0e9e0db3 100644
--- a/src/openvpn/multi.c
+++ b/src/openvpn/multi.c
@@ -1729,7 +1729,7 @@  multi_client_connect_mda(struct multi_context *m,
     {
         struct buffer_entry *be;
 
-        for (be = config->head; be != NULL; be = be->next)
+        for (be = mi->cc_config->head; be != NULL; be = be->next)
         {
             const char *opt = BSTR(&be->buf);
             options_string_import(&mi->context.options,