WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 8 months ago

#16906 closed defect (bug) (wontfix)

Bogus call to wp_kses_no_null()

Reported by: hakre Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: General Keywords: has-patch
Focuses: Cc:

Description

Nullchars and \ have been already replaced in string in question one line above. See wp_kses_no_null().

Attachments (1)

16906.patch (523 bytes) - added by hakre 3 years ago.
Remove Bogus Code

Download all attachments as: .zip

Change History (3)

hakre3 years ago

Remove Bogus Code

comment:1 hakre3 years ago

Small one, ran over it while on Related: #16903

comment:2 dd328 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

[3992] - It's there as extra protection in the event that a null slips through due to a PHP bug or charset edgecase (the latter being more likely)

As it's causing no harm, nor is it within code which is called a significant amount of times per page, lets not change it for the sake of changing it.

Note: See TracTickets for help on using tickets.