Make WordPress Core

Opened 11 months ago

Last modified 10 days ago

#63141 new defect (bug)

Getting Detected usage of a non-sanitized input variable error into the WordPress core files.

Reported by: viralsampat's profile viralsampat Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: General Keywords: dev-feedback
Focuses: coding-standards Cc:

Description

Hello Team,

I have checked wp-admin WordPress core files and I have found this "Detected usage of a non-sanitized input variable" error for few files.

Here, I have listed files:

src/wp-admin/includes/class-wp-ms-sites-list-table.php
src/wp-admin/includes/class-wp-privacy-requests-table.php
src/wp-admin/includes/class-wp-terms-list-table.php
src/wp-admin/includes/class-wp-theme-install-list-table.php
src/wp-admin/includes/class-wp-themes-list-table.php
src/wp-admin/includes/class-wp-users-list-table.php
src/wp-admin/includes/post.php

I have tested this into the WordPress 6.8-beta1.

Thanks,

Attachments (5)

63141.patch (7.3 KB) - added by viralsampat 11 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.
63141.2.patch (10.1 KB) - added by viralsampat 11 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.
63141.3.patch (8.9 KB) - added by viralsampat 10 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.
63141.4.patch (4.5 KB) - added by viralsampat 9 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.
63141.5.patch (4.4 KB) - added by viralsampat 8 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.

Download all attachments as: .zip

Change History (6)

@viralsampat
11 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

@viralsampat
11 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

@viralsampat
10 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

@viralsampat
9 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

@viralsampat
8 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

#1 @sajib1223
10 days ago

  • Keywords needs-testing removed

When I try to apply the patch https://core.trac.wordpress.org/attachment/ticket/63141/63141.5.patch against the latest trunk 7.0-alpha-61215-src, it fails with the following message:

Running "patch:63141" (patch) task
? Please select a patch to apply 63141.5.patch​ (4.4 KB) - added by viralsampat 8 months ago.
patching file src/wp-includes/class-wp-ajax-response.php
patching file src/wp-includes/class-wp-application-passwords.php
patching file src/wp-includes/ms-deprecated.php
patching file src/wp-includes/rest-api/class-wp-rest-server.php
Hunk #2 FAILED at 365.
1 out of 2 hunks FAILED -- saving rejects to file src/wp-includes/rest-api/class-wp-rest-server.php.rej
patching file src/wp-includes/user.php
Hunk #2 succeeded at 3208 (offset 26 lines).
Hunk #3 succeeded at 3338 (offset 26 lines).

The patch is failing to apply.

Note: See TracTickets for help on using tickets.