X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=img_auth.php;h=d7125897c5f9941e629bb361aa961067ec3888ac;hb=9196a9cf0803d47d5a8cf29cd7b8ea135425fda0;hp=b99773b7769b0f1425de966e3cd4512dc9ed0a4c;hpb=5fde23e6acda50246b30675fcf46a490082d30b2;p=lhc%2Fweb%2Fwiklou.git diff --git a/img_auth.php b/img_auth.php index b99773b776..d7125897c5 100644 --- a/img_auth.php +++ b/img_auth.php @@ -1,5 +1,4 @@ getText( 'path' ); - if( !$path ) { - wfForbidden( 'img-auth-accessdenied', 'img-auth-nopathinfo' ); - } -} else { - $path = $_SERVER['PATH_INFO']; -} +wfImageAuthMain(); +wfLogProfilingData(); + +function wfImageAuthMain() { + global $wgImgAuthPublicTest, $wgRequest; -$filename = realpath( $wgUploadDirectory . '/' . $path ); -$realUpload = realpath( $wgUploadDirectory ); + // See if this is a public Wiki (no protections). + if ( $wgImgAuthPublicTest + && in_array( 'read', User::getGroupPermissions( array( '*' ) ), true ) ) + { + // This is a public wiki, so disable this script (for private wikis only) + wfForbidden( 'img-auth-accessdenied', 'img-auth-public' ); + return; + } -// Basic directory traversal check -if( substr( $filename, 0, strlen( $realUpload ) ) != $realUpload ) - wfForbidden('img-auth-accessdenied','img-auth-notindir'); + // Get the requested file path (source file or thumbnail) + $matches = WebRequest::getPathInfo(); + if ( !isset( $matches['title'] ) ) { + wfForbidden( 'img-auth-accessdenied', 'img-auth-nopathinfo' ); + return; + } + $path = $matches['title']; + if ( $path && $path[0] !== '/' ) { + // Make sure $path has a leading / + $path = "/" . $path; + } -// Extract the file name and chop off the size specifier -// (e.g. 120px-Foo.png => Foo.png) -$name = wfBaseName( $path ); -if( preg_match( '!\d+px-(.*)!i', $name, $m ) ) - $name = $m[1]; + // Check for bug 28235: QUERY_STRING overriding the correct extension + $whitelist = array(); + $dotPos = strrpos( $path, '.' ); + if ( $dotPos !== false ) { + $whitelist[] = substr( $path, $dotPos + 1 ); + } + if ( !$wgRequest->checkUrlExtension( $whitelist ) ) { + return; + } -// Check to see if the file exists -if( !file_exists( $filename ) ) - wfForbidden('img-auth-accessdenied','img-auth-nofile',$filename); + // Get the local file repository + $repo = RepoGroup::singleton()->getRepo( 'local' ); -// Check to see if tried to access a directory -if( is_dir( $filename ) ) - wfForbidden('img-auth-accessdenied','img-auth-isdir',$filename); + // 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 ( !$repo->fileExists( $filename ) ) { + wfForbidden( 'img-auth-accessdenied','img-auth-nofile', $filename ); + return; + } -$title = Title::makeTitleSafe( NS_FILE, $name ); + $title = Title::makeTitleSafe( NS_FILE, $name ); + if ( !$title instanceof Title ) { // files have valid titles + wfForbidden( 'img-auth-accessdenied', 'img-auth-badtitle', $name ); + return; + } -// See if could create the title object -if( !$title instanceof Title ) - wfForbidden('img-auth-accessdenied','img-auth-badtitle',$name); + // Run hook for extension authorization plugins + if ( !wfRunHooks( 'ImgAuthBeforeStream', array( &$title, &$path, &$name, &$result ) ) ) { + wfForbidden( $result[0], $result[1], array_slice( $result, 2 ) ); + return; + } -// Run hook -if (!wfRunHooks( 'ImgAuthBeforeStream', array( &$title, &$path, &$name, &$result ) ) ) - wfForbidden($result[0],$result[1],array_slice($result,2)); - -// Check user authorization for this title -// UserCanRead Checks Whitelist too -if( !$title->userCanRead() ) - wfForbidden('img-auth-accessdenied','img-auth-noread',$name); + // Check user authorization for this title + // 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."`." ); -wfStreamFile( $filename, array( 'Cache-Control: private', 'Vary: Cookie' ) ); -wfLogProfilingData(); + // Stream the requested file + wfDebugLog( 'img_auth', "Streaming `".$filename."`." ); + $repo->streamFile( $filename, array( 'Cache-Control: private', 'Vary: Cookie' ) ); +} /** * Issue a standard HTTP 403 Forbidden header ($msg1-a message index, not a message) and an * error message ($msg2, also a message index), (both required) then end the script - * subsequent arguments to $msg2 will be passed as parameters only for replacing in $msg2 + * subsequent arguments to $msg2 will be passed as parameters only for replacing in $msg2 + * @param $msg1 + * @param $msg2 */ -function wfForbidden($msg1,$msg2) { +function wfForbidden( $msg1, $msg2 ) { global $wgImgAuthDetails; + $args = func_get_args(); array_shift( $args ); array_shift( $args ); - $MsgHdr = htmlspecialchars(wfMsg($msg1)); - $detailMsg = (htmlspecialchars(wfMsg(($wgImgAuthDetails ? $msg2 : 'badaccess-group0'),$args))); - wfDebugLog('img_auth', "wfForbidden Hdr:".wfMsgExt( $msg1, array('language' => 'en'))." Msg: ". - wfMsgExt($msg2,array('language' => 'en'),$args)); + + $msgHdr = htmlspecialchars( wfMsg( $msg1 ) ); + $detailMsgKey = $wgImgAuthDetails ? $msg2 : 'badaccess-group0'; + $detailMsg = htmlspecialchars( wfMsg( $detailMsgKey, $args ) ); + + wfDebugLog( 'img_auth', + "wfForbidden Hdr:" . wfMsgExt( $msg1, array( 'language' => 'en' ) ). " Msg: ". + wfMsgExt( $msg2, array( 'language' => 'en' ), $args ) + ); + header( 'HTTP/1.0 403 Forbidden' ); header( 'Cache-Control: no-cache' ); header( 'Content-Type: text/html; charset=utf-8' ); echo << -

$MsgHdr

+

$msgHdr

$detailMsg

ENDS; - wfLogProfilingData(); - exit(); }