[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