From 966971e58914ef254595532118d37dd604231a5c Mon Sep 17 00:00:00 2001
From: Michael Tremer <michael.tremer@ipfire.org>
Date: Tue, 25 Apr 2017 12:32:05 +0100
Subject: [PATCH] captive: Don't compare action string

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---
 html/cgi-bin/captive/index.cgi  | 6 +++---
 html/html/captive/template.html | 1 +
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/html/cgi-bin/captive/index.cgi b/html/cgi-bin/captive/index.cgi
index 8067f25d97..05619b47c3 100755
--- a/html/cgi-bin/captive/index.cgi
+++ b/html/cgi-bin/captive/index.cgi
@@ -53,8 +53,8 @@ unless (-f $clients){ system("touch $clients"); }
 #Read settings
 &General::readhash("$settingsfile", \%settings) if(-f $settingsfile);
 
-#Actions
-if ($cgiparams{'ACTION'} eq "$Lang::tr{'gpl i accept these terms and conditions'}"){
+# Actions
+if ($cgiparams{'ACTION'} eq "SUBMIT") {
 	#Get Clients IP-Address
 	my $ip_address = $ENV{X_FORWARDED_FOR} || $ENV{REMOTE_ADDR} ||"";
 
@@ -82,7 +82,7 @@ if ($cgiparams{'ACTION'} eq "$Lang::tr{'gpl i accept these terms and conditions'
 	}	
 }
 
-if ($cgiparams{'ACTION'} eq "$Lang::tr{'Captive activate'}"){
+if ($cgiparams{'ACTION'} eq "SUBMIT") {
 	my $ip_address;
 	my $mac_address;
 	my $granted=0;
diff --git a/html/html/captive/template.html b/html/html/captive/template.html
index 26f4142089..369532a084 100644
--- a/html/html/captive/template.html
+++ b/html/html/captive/template.html
@@ -30,6 +30,7 @@
 							</TMPL_IF>
 
 							<form action="" method="POST">
+								<input type="hidden" name="ACTION" value="SUBMIT">
 								<input type="hidden" name="url" value="<TMPL_VAR NAME="URL">">
 
 								<TMPL_IF NAME="VOUCHER">
-- 
2.39.2