[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