Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #53271


Ignore:
Timestamp:
05/25/2021 09:58:06 AM (5 years ago)
Author:
SergeyBiryukov
Comment:

Hi there, welcome back to WordPress Trac! Thanks for the report.

This has come up a few times before, for example in #36177, #30806, #44700, #45773, and most recently in #48049.

Per the Security FAQ, these errors are considered a server configuration issue rather than a security issue:

Why are there path disclosures when directly loading certain files? This is a server configuration problem. Never enable display_errors on a production site.

Instead of changing all PHP files to add a check for ABSPATH, this should be done in a central location, which is being discussed in #36177.

Let's continue the discussion in that ticket, as it would be best to keep all discussion on the general idea of "blocking malicious requests" in a single place, even if any implemented change does not necessarily follow how the ticket originally intended on it being implemented.

That said, it looks like this ticket is more about unnecessary error logging rather than a path disclosure. Still, blocking direct access to the files in question using the web server configuration file should resolve the issue for now, until any changes are implemented in core.

See also a related similar ticket for bundled themes: #47154.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #53271

    • Property Status changed from new to closed
    • Property Component changed from I18N to General
    • Property Resolution changed from to duplicate
    • Property Milestone changed from Awaiting Review to
  • Ticket #53271 – Description

    initial v1  
    33I am getting 500 error due to WordPress loopholes,
    44if someone trying to access directly this file then it's generating 500 error because in these below file wp extend another class "Walker" without checking ABSPATH
    5 
     5{{{
    66if ( ! defined( 'ABSPATH' ) ) {
    77        exit; // Exit if accessed directly
    88    }
    9 
     9}}}
    1010File: wp-includes/class-walker-nav-menu.php
    1111File: wp-includes/class-walker-comment.php