captive: Verify that the user actually accepted the terms and conditions

[people/pmueller/ipfire-2.x.git] / html / cgi-bin / captive / index.cgi

diff --git a/html/cgi-bin/captive/index.cgi b/html/cgi-bin/captive/index.cgi
index 482ea9016be22118d6263907a34a67ad15f31040..b891255744a946b9b462b3624ad9f21d77848621 100755 (executable)
--- a/html/cgi-bin/captive/index.cgi
+++ b/html/cgi-bin/captive/index.cgi
@@ -116,15 +116,22 @@ if ($cgiparams{'ACTION'} eq "SUBMIT") {
                        $errormessage = $Lang::tr{"Captive please enter a coupon code"};
                }
 
-       # License
+       # Terms
        } else {
-               # Copy session expiry time
-               $clientshash{$key}[3] = $settings{'SESSION_TIME'} || "0";
+               # Make sure that they have been accepted
+               if ($cgiparams{'TERMS'} eq "on") {
+                       # Copy session expiry time
+                       $clientshash{$key}[3] = $settings{'SESSION_TIME'} || "0";
 
-               # No coupon code
-               $clientshash{$key}[4] = "TERMS";
+                       # No coupon code
+                       $clientshash{$key}[4] = "TERMS";
 
-               &General::log("Captive", "Internet access granted via license agreement for $ip_address until $clientshash{$key}[3]");
+                       &General::log("Captive", "Internet access granted via license agreement for $ip_address until $clientshash{$key}[3]");
+
+               # The terms have not been accepted
+               } else {
+                       $errormessage = $Lang::tr{'Captive please accept the terms and conditions'};
+               }
        }
 
        # If no errors were found, save configruation and reload