From a21c222f39689b70baa92cb1267b1b024507cc0e Mon Sep 17 00:00:00 2001 From: chrisaga Date: Sun, 16 Apr 2006 10:53:01 +0000 Subject: - security : add some control to download url. Somebody sould take it over for other controls git-svn-id: http://piwigo.org/svn/trunk@1187 68402e56-0260-453c-a942-63ccdbb3a9ee --- action.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'action.php') diff --git a/action.php b/action.php index ef635a4d1..c20b07d3a 100644 --- a/action.php +++ b/action.php @@ -65,8 +65,11 @@ function force_download ($filename) //--------------------------------------------------------- download big picture if ( isset( $_GET['dwn'] ) ) { -//TODO : verify the path begins with './gallerie' and doesn't contains any '..' -// in order to avoid hacking atempts +//TODO : verify the path begins with something in galleries_url and that user has access rights to the picture +// in order to avoid hacking atempts by forged url + if (preg_match('/\.\./',$_GET['dwn'])) { + die('Hacking attempt!'); + } force_download($_GET['dwn']); } -- cgit v1.2.3