From: Fabian Knittel <fabian.knit...@lettink.de>
This patch moves multi_client_connect_setenv into
multi_client_connect_early_setup and makes sure that every client-connect
handling function updates the virtual address selection.
Background: This unifies how the client-connect handling functions work.
Signed-off-by: Fabian Knittel <fabian.knit...@lettink.de>
Signed-off-by: Arne Schwabe <a...@rfc2549.org>
Patch V5: Rebase on master
Signed-off-by: Arne Schwabe <a...@rfc2549.org>
---
src/openvpn/multi.c | 22 +++++++++++++---------
1 file changed, 13 insertions(+), 9 deletions(-)
diff --git a/src/openvpn/multi.c b/src/openvpn/multi.c
index 35e0bd10..539ebfc0 100644
--- a/src/openvpn/multi.c
+++ b/src/openvpn/multi.c
@@ -2150,6 +2150,12 @@ multi_client_connect_early_setup(struct multi_context *m,
/* reset pool handle to null */
mi->vaddr_handle = -1;
+
+ /* do --client-connect setenvs */
+ multi_select_virtual_addr(m, mi);
+
+ multi_client_connect_setenv(m, mi);
+
}
/**
@@ -2195,6 +2201,13 @@ multi_client_connect_source_ccd(struct multi_context *m,
CLIENT_CONNECT_OPT_MASK,
option_types_found,
mi->context.c2.es);
+ /*
+ * Select a virtual address from either --ifconfig-push in
+ * --client-config-dir file or --ifconfig-pool.
+ */
+ multi_select_virtual_addr(m, mi);
+
+ multi_client_connect_setenv(m, mi);
}
gc_free(&gc);
}
@@ -2236,15 +2249,6 @@ multi_connection_established(struct multi_context *m,
struct multi_instance *mi)
multi_client_connect_source_ccd(m, mi, &option_types_found);
- /*
- * Select a virtual address from either --ifconfig-push in
- * --client-config-dir file or --ifconfig-pool.
- */
- multi_select_virtual_addr(m, mi);
-
- /* do --client-connect setenvs */
- multi_client_connect_setenv(m, mi);
-
multi_client_connect_call_plugin_v1(m, mi, &option_types_found,
&cc_succeeded,
&cc_succeeded_count);
--
2.26.2
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
