X-Git-Url: http://git.cyclocoop.org/?a=blobdiff_plain;f=img_auth.php;h=d7125897c5f9941e629bb361aa961067ec3888ac;hb=92604f6870bcc378b8a7ab1876c0e70f41e7e672;hp=fc431f8b617779debe609ebb4248f93d5dc3fb18;hpb=f8d6b31beea91fa79dbdcb93dd161426a457ba3f;p=lhc%2Fweb%2Fwiklou.git diff --git a/img_auth.php b/img_auth.php index fc431f8b61..d7125897c5 100644 --- a/img_auth.php +++ b/img_auth.php @@ -1,5 +1,4 @@ getRepo( 'local' ); - // Basic directory traversal check - if ( substr( $filename, 0, strlen( $realUpload ) ) != $realUpload ) { - wfForbidden( 'img-auth-accessdenied', 'img-auth-notindir' ); - return; + // Get the full file storage path and extract the source file name. + // (e.g. 120px-Foo.png => Foo.png or page2-120px-Foo.png => Foo.png). + // This only applies to thumbnails, and all thumbnails should + // be under a folder that has the source file name. + if ( strpos( $path, '/thumb/' ) === 0 ) { + $name = wfBaseName( dirname( $path ) ); // file is a thumbnail + $filename = $repo->getZonePath( 'thumb' ) . substr( $path, 6 ); // strip "/thumb" + } else { + $name = wfBaseName( $path ); // file is a source file + $filename = $repo->getZonePath( 'public' ) . $path; } // Check to see if the file exists - if ( !file_exists( $filename ) ) { + if ( !$repo->fileExists( $filename ) ) { wfForbidden( 'img-auth-accessdenied','img-auth-nofile', $filename ); return; } - - // Check to see if tried to access a directory - if ( is_dir( $filename ) ) { - wfForbidden( 'img-auth-accessdenied','img-auth-isdir', $filename ); - return; - } - - // Extract the file name and chop off the size specifier - // (e.g. 120px-Foo.png => Foo.png or page2-120px-Foo.png => Foo.png). - // This only applies to thumbnails, and all thumbnails should have - // a width indicator and be under a folder that has the source file name. - $name = wfBaseName( $path ); - if ( preg_match( '!(?:[^-]*-)*?\d+px-(.*)!i', $name ) ) { - $name = wfBaseName( dirname( $path ) ); // this file is a thumbnail - } $title = Title::makeTitleSafe( NS_FILE, $name ); if ( !$title instanceof Title ) { // files have valid titles @@ -114,17 +121,17 @@ function wfImageAuthMain() { wfForbidden( $result[0], $result[1], array_slice( $result, 2 ) ); return; } - + // Check user authorization for this title - // UserCanRead Checks Whitelist too - if ( !$title->userCanRead() ) { + // Checks Whitelist too + if ( !$title->userCan( 'read' ) ) { wfForbidden( 'img-auth-accessdenied', 'img-auth-noread', $name ); return; } // Stream the requested file wfDebugLog( 'img_auth', "Streaming `".$filename."`." ); - StreamFile::stream( $filename, array( 'Cache-Control: private', 'Vary: Cookie' ) ); + $repo->streamFile( $filename, array( 'Cache-Control: private', 'Vary: Cookie' ) ); } /**