[wp-trac] [WordPress Trac] #9930: is_serialized() returns false on serialized doubles

WordPress Trac wp-trac at lists.automattic.com
Tue Dec 22 13:16:27 UTC 2009


#9930: is_serialized() returns false on serialized doubles
---------------------------------+------------------------------------------
 Reporter:  vladimir_kolesnikov  |        Owner:  westi   
     Type:  defect (bug)         |       Status:  reopened
 Priority:  normal               |    Milestone:  3.0     
Component:  General              |      Version:          
 Severity:  minor                |   Resolution:          
 Keywords:  has-patch            |  
---------------------------------+------------------------------------------

Comment(by Denis-de-Bernardy):

 re-opening with a new patch. the new regex makes it much slower, and it
 can run into backtracking problems with large strings.

 if we are to keep it, this:

 {{{
 ^(N;)|([aOs]:[0-9]+:.*[;}])|([bid]:[0-9.E+-]+;)$
 }}}

 should be:

 {{{
 ^(?:N;|[aOs]:[0-9]+:.*[;}]|[bid]:[0-9.E+-]+;)$
 }}}

 else is_serialized may return true for garbage.

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


More information about the wp-trac mailing list