summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--etc/inc/captiveportal.inc1
-rw-r--r--usr/local/captiveportal/index.php4
2 files changed, 4 insertions, 1 deletions
diff --git a/etc/inc/captiveportal.inc b/etc/inc/captiveportal.inc
index 8e20ceb..0145962 100644
--- a/etc/inc/captiveportal.inc
+++ b/etc/inc/captiveportal.inc
@@ -915,6 +915,7 @@ function captiveportal_disconnect($dbent, $radiusservers,$term_cause = 1,$stop_t
function captiveportal_disconnect_client($sessionid, $term_cause = 1, $logoutReason = "LOGOUT") {
global $g, $config;
+ $sessionid = SQLite3::escapeString($sessionid);
$radiusservers = captiveportal_get_radius_servers();
/* read database */
diff --git a/usr/local/captiveportal/index.php b/usr/local/captiveportal/index.php
index a8b3842..eaaf920 100644
--- a/usr/local/captiveportal/index.php
+++ b/usr/local/captiveportal/index.php
@@ -137,7 +137,9 @@ setTimeout('window.close();',5000) ;
</html>
EOD;
- captiveportal_disconnect_client($_POST['logout_id']);
+
+ $safe_logout_id = SQLite3::escapeString($_POST['logout_id']);
+ captiveportal_disconnect_client($safe_logout_id);
} else if ($macfilter && $clientmac && captiveportal_blocked_mac($clientmac)) {
captiveportal_logportalauth($clientmac,$clientmac,$clientip,"Blocked MAC address");
OpenPOWER on IntegriCloud