From c68ac04020d85a296c784de18f8def3f365f9f6a Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <noreply@fakecake.org>
Date: Sun, 3 Mar 2019 07:53:42 +0300
Subject: login: only allow relative URLs in return=

---
 classes/handler/public.php | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

(limited to 'classes/handler/public.php')

diff --git a/classes/handler/public.php b/classes/handler/public.php
index c3f18d8bb..16f8020bb 100755
--- a/classes/handler/public.php
+++ b/classes/handler/public.php
@@ -712,8 +712,10 @@ class Handler_Public extends Handler {
 				user_error("Failed login attempt for $login from {$_SERVER['REMOTE_ADDR']}", E_USER_WARNING);
 			}
 
-			if (clean($_REQUEST['return'])) {
-				header("Location: " . clean($_REQUEST['return']));
+			$return = clean($_REQUEST['return']);
+
+			if ($return && !is_absolute_url($return)) {
+				header("Location: " . $return);
 			} else {
 				header("Location: " . get_self_url_prefix());
 			}
-- 
cgit v1.2.3