[wp-trac] [WordPress Trac] #17445: Give a speed kick to combined is_serialized() + is_serialized_string() + maybe_unserialize() usage

WordPress Trac wp-trac at lists.automattic.com
Sun May 15 14:17:46 UTC 2011


#17445: Give a speed kick to combined is_serialized() + is_serialized_string() +
maybe_unserialize() usage
--------------------------+------------------------------
 Reporter:  hakre         |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  General       |     Version:  3.2
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by hakre):

 Added a patch. Technically it would allow to deprecate
 is_serialized_string() as it is not needed any longer by core.

 Next to that I suggest to remove the trim() from is_serialized() as
 maybe_serialize() does not unserialize on trimmed but on untrimmed data.
 Technically it does fail on untrimmed data which has been reported as
 being serialized by is_serialize().

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/17445#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list