From 29b68a6b70b2b6ea7e7e118f20c051bec147cc00 Mon Sep 17 00:00:00 2001 From: Eli Bosley Date: Wed, 21 May 2025 16:22:34 -0400 Subject: [PATCH] Update GUIModeSessionFix.php Fix: this will fix a rare edge case that a function may be "defined twice", breaking the entire webGUI. --- .../include/DefaultPageLayout/GUIModeSessionFix.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/emhttp/plugins/dynamix/include/DefaultPageLayout/GUIModeSessionFix.php b/emhttp/plugins/dynamix/include/DefaultPageLayout/GUIModeSessionFix.php index 516284e1e..6024e62d1 100644 --- a/emhttp/plugins/dynamix/include/DefaultPageLayout/GUIModeSessionFix.php +++ b/emhttp/plugins/dynamix/include/DefaultPageLayout/GUIModeSessionFix.php @@ -4,16 +4,16 @@ * This can be deleted if GUI mode authentication is enabled. */ -function is_localhost() +function is_localhost_gui() { // Use the peer IP, not the Host header which can be spoofed return $_SERVER['REMOTE_ADDR'] === '127.0.0.1' || $_SERVER['REMOTE_ADDR'] === '::1'; } -function is_good_session() +function is_good_session_gui() { return isset($_SESSION) && isset($_SESSION['unraid_user']) && isset($_SESSION['unraid_login']); } -if (is_localhost() && !is_good_session()) { +if (is_localhost_gui() && !is_good_session_gui()) { if (session_status() === PHP_SESSION_ACTIVE) { session_destroy(); } @@ -23,4 +23,4 @@ if (is_localhost() && !is_good_session()) { session_write_close(); my_logger("Unraid GUI-boot: created root session for localhost request."); } -?> \ No newline at end of file +?>