Networks and Sites: check if the BLOGUPLOADDIR constant is defined before using it.

This change prevents a fatal PHP error from occurring when `ms-files.php` is accessed without legacy multisite file-handling support being configured.

Instead of the fatal error, these same requests will new return a header status `500` along with a `die()` message "500 - Directory not configured." This differs from the existing `404` statuses to help indicate to site owners this fallback is not working as intended.

Props danielpw, dd32, jakeparis, johnjamesjacoby, spacedmonkey, saurabh.dhariwal, sukhendu2002, realloc.

Fixes #63104.
Built from https://develop.svn.wordpress.org/trunk@60956


git-svn-id: http://core.svn.wordpress.org/trunk@60292 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
johnjamesjacoby
2025-10-17 17:14:32 +00:00
parent 737dbdd72c
commit 5f14fc0a16
2 changed files with 6 additions and 1 deletions

View File

@@ -25,6 +25,11 @@ if ( '1' === $current_blog->archived || '1' === $current_blog->spam || '1' === $
die( '404 — File not found.' );
}
if ( ! defined( 'BLOGUPLOADDIR' ) ) {
status_header( 500 );
die( '500 — Directory not configured.' );
}
$file = rtrim( BLOGUPLOADDIR, '/' ) . '/' . str_replace( '..', '', $_GET['file'] );
if ( ! is_file( $file ) ) {
status_header( 404 );

View File

@@ -16,7 +16,7 @@
*
* @global string $wp_version
*/
$wp_version = '6.9-alpha-60955';
$wp_version = '6.9-alpha-60956';
/**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.