0000555: serialize for array values add an unneeded ";" to the serialized value (and unserialize need it)

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0000555

[Quercus]

minor

always

12-23-05 07:39

12-23-05 08:49

Reporter

bago

View Status

@0@

Assigned To

Priority

normal

Resolution

fixed

Status

closed

Product Version

Summary

0000555: serialize for array values add an unneeded ";" to the serialized value (and unserialize need it)

Description

<?php
print(serialize(array("a")));
?>

in official php: a:1:{i:0;s:1:"a";}
in quercus: a:1:{i:0;s:1:"a";};

so quercus unserialize is not able to unserialize data serialized by official php

See the fix in the "additional note section"

Additional Information

Here is the fix:

QuercusVariableModule:666:
put
    unserializeExpect(is, '}');
instead of
    unserializeExpect(is, '}');
    unserializeExpect(is, ';');

ArrayValue:589:
put
    sb.append("}");
instead of
    sb.append("};");

Attached Files

Relationships

Notes
(0000565) ferg 12-23-05 08:49	php/1215, php/1225

Issue History
Date Modified	Username	Field	Change
12-23-05 07:39	bago	New Issue
12-23-05 08:49	ferg	Status	new => closed
12-23-05 08:49	ferg	Note Added: 0000565
12-23-05 08:49	ferg	Resolution	open => fixed
12-23-05 08:49	ferg	Fixed in Version	=> 3.0.18

Mantis 1.0.0rc3[^]

29 total queries executed.
26 unique queries executed.