[wp-trac] [WordPress Trac] #29557: PHP Crash on Large Block of HTML
WordPress Trac
noreply at wordpress.org
Fri Sep 12 09:39:38 UTC 2014
#29557: PHP Crash on Large Block of HTML
-----------------------------------+--------------------
Reporter: MrBobDobolina | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.0.1
Component: Formatting | Version: 4.0
Severity: major | Resolution:
Keywords: wptexturize has-patch | Focuses:
-----------------------------------+--------------------
Comment (by miqrogroove):
Updated summary of miqro-29557.5.patch:
* Expand the wptexturize regexp to include the list of registered
shortcodes.
* Avoid backtracking after {{{[}}} chars by not filtering params in
registered shortcodes.
* This will cause escaped shortcodes and their params to become texturized
if not registered.
* Registered shortcode params will never be texturized, even when escaped.
* Move all tests involving unregistered shortcodes to a new and improved
unit.
* Update one test involving HTML within shortcode params.
Tests for dangling angle braces within shortcode params have not been
added. This feature is not even documented AFAIK. In the Codex, HTML is
only mentioned as allowed in between shortcode tags.
Decided not to remove the check on escaped shortcodes. There are too many
potential corner cases involving the different regexps for texturize and
shortcodes. Instead, I simply nested the if statement to help speed up
shortcode processing.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29557#comment:42>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list