Fix multiple configured scripts conflicts issue

PATCH New David Sommerseth 1 participant 1 message Jul 13, 2010

Share & tools

David Sommerseth

author

2010-07-13 13:32 UTC #0 lore raw

PATCH 1/1 Fix multiple configured scripts conflicts issue

This is a fix for trac ticket #20,
<https://community.openvpn.net/openvpn/ticket/20>
which was started in the sf.net bug tracker:
<http://sourceforge.net/tracker/?func=detail&aid=2078470&group_id=48978&atid=454719>

The implemented solution is to give a warning for each of the different script hooks available. The last configured script will override any earlier configured scripts, to ensure that the command line can override the configuration file.

Signed-off-by David Sommerseth <[email protected]...>

---
 options.c |   36 ++++++++++++++++++++++++++++++++++++
 1 files changed, 36 insertions(+), 0 deletions(-)

options.c +36 -0

index b78158e..9bbb51e 100644
@@ -3877,6 +3877,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->ipchange) {
+	msg (M_WARN, "Multiple --ipchange scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->ipchange = string_substitute (p[1], ',', ' ', &options->gc);
     }
   else if (streq (p[0], "float"))

@@ -3923,6 +3927,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->up_script) {
+	msg (M_WARN, "Multiple --up scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->up_script = p[1];
     }
   else if (streq (p[0], "down") && p[1])

@@ -3930,6 +3938,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->down_script) {
+	msg (M_WARN, "Multiple --down scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->down_script = p[1];
     }
   else if (streq (p[0], "down-pre"))

@@ -4598,6 +4610,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->route_script) {
+	msg (M_WARN, "Multiple --route-up scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->route_script = p[1];
     }
   else if (streq (p[0], "route-noexec"))

@@ -4927,6 +4943,10 @@ add_option (struct options *options,
 	  msg (msglevel, "--auth-user-pass-verify requires a second parameter ('via-env' or 'via-file')");
 	  goto err;
 	}
+      if (options->auth_user_pass_verify_script) {
+	msg (M_WARN, "Multiple --auth-user-pass-verify scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->auth_user_pass_verify_script = p[1];
     }
   else if (streq (p[0], "client-connect") && p[1])

@@ -4934,6 +4954,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->client_connect_script) {
+	msg (M_WARN, "Multiple --client-connect scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->client_connect_script = p[1];
     }
   else if (streq (p[0], "client-disconnect") && p[1])

@@ -4941,6 +4965,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->client_disconnect_script) {
+	msg (M_WARN, "Multiple --client-disconnect scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->client_disconnect_script = p[1];
     }
   else if (streq (p[0], "learn-address") && p[1])

@@ -4948,6 +4976,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->learn_address_script) {
+	msg (M_WARN, "Multiple --learn-address scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->learn_address_script = p[1];
     }
   else if (streq (p[0], "tmp-dir") && p[1])

@@ -5708,6 +5740,10 @@ add_option (struct options *options,
       VERIFY_PERMISSION (OPT_P_SCRIPT);
       if (!no_more_than_n_args (msglevel, p, 2, NM_QUOTE_HINT))
 	goto err;
+      if (options->tls_verify) {
+	msg (M_WARN, "Multiple --tls-verify scripts defined.  "
+	     "The previously configured script is overridden.");
+      }
       options->tls_verify = string_substitute (p[1], ',', ' ', &options->gc);
     }
   else if (streq (p[0], "tls-remote") && p[1])

Signature · 1 line

1.7.1.1