My proposed fix to bug 34987: gender not working in many special pages.
[lhc/web/wiklou.git] / img_auth.php
index bc4464d..3999bf3 100644 (file)
  *
  * Optional Parameters
  *
- * - Set $wgImgAuthDetails = true if you want the reason the access was denied messages to be displayed 
- *       instead of just the 403 error (doesn't work on IE anyway),  otherwise will only appear in error logs
+ * - Set $wgImgAuthDetails = true if you want the reason the access was denied messages to
+ *       be displayed instead of just the 403 error (doesn't work on IE anyway),
+ *       otherwise it will only appear in error logs
  * - Set $wgImgAuthPublicTest false if you don't want to just check and see if all are public
  *       must be set to false if using specific restrictions such as LockDown or NSFileRepo
  *
- *  For security reasons, you usually don't want your user to know *why* access was denied, just that it was.
- *  If you want to change this, you can set $wgImgAuthDetails to 'true' in localsettings.php and it will give the user the reason
- *  why access was denied.
+ *  For security reasons, you usually don't want your user to know *why* access was denied,
+ *  just that it was. If you want to change this, you can set $wgImgAuthDetails to 'true'
+ *  in localsettings.php and it will give the user the reason why access was denied.
  *
  * Your server needs to support PATH_INFO; CGI-based configurations usually don't.
  *
  **/
 
 define( 'MW_NO_OUTPUT_COMPRESSION', 1 );
-require_once( dirname( __FILE__ ) . '/includes/WebStart.php' );
+if ( isset( $_SERVER['MW_COMPILED'] ) ) {
+       require ( 'phase3/includes/WebStart.php' );
+} else {
+       require ( dirname( __FILE__ ) . '/includes/WebStart.php' );
+}
 wfProfileIn( 'img_auth.php' );
-require_once( dirname( __FILE__ ) . '/includes/StreamFile.php' );
 
-// See if this is a public Wiki (no protections)
-if ( $wgImgAuthPublicTest 
-       && in_array( 'read', User::getGroupPermissions( array( '*' ) ), true ) )
-{
-       wfForbidden('img-auth-accessdenied','img-auth-public');
-}
+# Set action base paths so that WebRequest::getPathInfo()
+# recognizes the "X" as the 'title' in ../image_auth/X urls.
+$wgArticlePath = false; # Don't let a "/*" article path clober our action path
+$wgActionPaths = array( "$wgUploadPath/" );
 
-// Extract path and image information
-if( !isset( $_SERVER['PATH_INFO'] ) )
-       wfForbidden('img-auth-accessdenied','img-auth-nopathinfo');
+wfImageAuthMain();
+wfLogProfilingData();
 
-$path = $_SERVER['PATH_INFO'];
-$filename = realpath( $wgUploadDirectory . $_SERVER['PATH_INFO'] );
-$realUpload = realpath( $wgUploadDirectory );
+function wfImageAuthMain() {
+       global $wgImgAuthPublicTest, $wgRequest;
 
-// Basic directory traversal check
-if( substr( $filename, 0, strlen( $realUpload ) ) != $realUpload )
-       wfForbidden('img-auth-accessdenied','img-auth-notindir');
+       // 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;
+       }
 
-// 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];
+       // 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;
+       }
 
-// Check to see if the file exists
-if( !file_exists( $filename ) )
-       wfForbidden('img-auth-accessdenied','img-auth-nofile',$filename);
+       // 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 tried to access a directory
-if( is_dir( $filename ) )
-       wfForbidden('img-auth-accessdenied','img-auth-isdir',$filename);
+       // Get the local file repository
+       $repo = RepoGroup::singleton()->getRepo( 'local' );
 
+       // 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;
+       }
 
-$title = Title::makeTitleSafe( NS_FILE, $name );
+       // Check to see if the file exists
+       if ( !$repo->fileExists( $filename, FileRepo::FILES_ONLY ) ) {
+               wfForbidden( 'img-auth-accessdenied','img-auth-nofile', $filename );
+               return;
+       }
 
-// See if could create the title object
-if( !$title instanceof Title ) 
-       wfForbidden('img-auth-accessdenied','img-auth-badtitle',$name);
+       $title = Title::makeTitleSafe( NS_FILE, $name );
+       if ( !$title instanceof Title ) { // files have valid titles
+               wfForbidden( 'img-auth-accessdenied', 'img-auth-badtitle', $name );
+               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);
+       // Run hook for extension authorization plugins
+       if ( !wfRunHooks( 'ImgAuthBeforeStream', array( &$title, &$path, &$name, &$result ) ) ) {
+               wfForbidden( $result[0], $result[1], array_slice( $result, 2 ) );
+               return;
+       }
 
-// Stream the requested file
-wfDebugLog( 'img_auth', "Streaming `".$filename."`." );
-wfStreamFile( $filename, array( 'Cache-Control: private', 'Vary: Cookie' ) );
-wfLogProfilingData();
+       // 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."`." );
+       $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 <<<ENDS
 <html>
 <body>
-<h1>$MsgHdr</h1>
+<h1>$msgHdr</h1>
 <p>$detailMsg</p>
 </body>
 </html>
 ENDS;
-       wfLogProfilingData();
-       exit();
 }