helly Wed Mar 9 18:58:45 2005 EDT Modified files: /php-src/ext/standard var_unserializer.c Log: - Fix #31442 unserialize broken on 64-bit systems - Fix one warning http://cvs.php.net/diff.php/php-src/ext/standard/var_unserializer.c?r1=1.60&r2=1.61&ty=u Index: php-src/ext/standard/var_unserializer.c diff -u php-src/ext/standard/var_unserializer.c:1.60 php-src/ext/standard/var_unserializer.c:1.61 --- php-src/ext/standard/var_unserializer.c:1.60 Tue Mar 8 01:40:05 2005 +++ php-src/ext/standard/var_unserializer.c Wed Mar 9 18:58:44 2005 @@ -1,5 +1,5 @@ -/* Generated by re2c 0.9.4 on Tue Mar 8 08:39:34 2005 */ -#line 1 "/usr/src/php/php5/ext/standard/var_unserializer.re" +/* Generated by re2c 0.9.4 on Thu Mar 10 00:54:25 2005 */ +#line 1 "/usr/src/php-cvs/ext/standard/var_unserializer.re" /* +----------------------------------------------------------------------+ | PHP Version 5 | @@ -18,7 +18,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: var_unserializer.c,v 1.60 2005/03/08 06:40:05 sniper Exp $ */ +/* $Id: var_unserializer.c,v 1.61 2005/03/09 23:58:44 helly Exp $ */ #include "php.h" #include "ext/standard/php_var.h" @@ -29,7 +29,7 @@ typedef struct { zval *data[VAR_ENTRIES_MAX]; - int used_slots; + long used_slots; void *next; } var_entries; @@ -82,7 +82,7 @@ PHPAPI void var_replace(php_unserialize_data_t *var_hashx, zval *ozval, zval **nzval) { - int i; + long i; var_entries *var_hash = var_hashx->first; while (var_hash) { @@ -96,7 +96,7 @@ } } -static int var_access(php_unserialize_data_t *var_hashx, int id, zval ***store) +static int var_access(php_unserialize_data_t *var_hashx, long id, zval ***store) { var_entries *var_hash = var_hashx->first; @@ -117,7 +117,7 @@ PHPAPI void var_destroy(php_unserialize_data_t *var_hashx) { void *next; - int i; + long i; var_entries *var_hash = var_hashx->first; while (var_hash) { @@ -147,15 +147,15 @@ #define YYMARKER marker -#line 155 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 155 "/usr/src/php-cvs/ext/standard/var_unserializer.re" -static inline int parse_iv2(const unsigned char *p, const unsigned char **q) +static inline long parse_iv2(const unsigned char *p, const unsigned char **q) { char cursor; - int result = 0; + long result = 0; int neg = 0; switch (*p) { @@ -180,7 +180,7 @@ return result; } -static inline int parse_iv(const unsigned char *p) +static inline long parse_iv(const unsigned char *p) { return parse_iv2(p, NULL); } @@ -210,7 +210,7 @@ #define UNSERIALIZE_PARAMETER zval **rval, const unsigned char **p, const unsigned char *max, php_unserialize_data_t *var_hash TSRMLS_DC #define UNSERIALIZE_PASSTHRU rval, p, max, var_hash TSRMLS_CC -static inline int process_nested_data(UNSERIALIZE_PARAMETER, HashTable *ht, int elements) +static inline int process_nested_data(UNSERIALIZE_PARAMETER, HashTable *ht, long elements) { while (elements-- > 0) { zval *key, *data, **old_data; @@ -279,7 +279,7 @@ static inline int object_custom(UNSERIALIZE_PARAMETER, zend_class_entry *ce) { - int datalen; + long datalen; if(ce->unserialize == NULL) { zend_error(E_WARNING, "Class %s has no unserializer", ce->name); @@ -291,7 +291,7 @@ (*p) += 2; if(datalen < 0 || (*p) + datalen >= max) { - zend_error(E_WARNING, "Insufficient data for unserializing - %d required, %d present", datalen, max - (*p)); + zend_error(E_WARNING, "Insufficient data for unserializing - %ld required, %d present", datalen, max - (*p)); return 0; } @@ -304,10 +304,10 @@ return finish_nested_data(UNSERIALIZE_PASSTHRU); } -static inline int object_common1(UNSERIALIZE_PARAMETER, zend_class_entry *ce) +static inline long object_common1(UNSERIALIZE_PARAMETER, zend_class_entry *ce) { - int elements; - + long elements; + elements = parse_iv2((*p) + 2, p); (*p) += 2; @@ -316,7 +316,7 @@ return elements; } -static inline int object_common2(UNSERIALIZE_PARAMETER, int elements) +static inline int object_common2(UNSERIALIZE_PARAMETER, long elements) { zval *retval_ptr = NULL; zval fname; @@ -448,7 +448,7 @@ if(yych == ':') goto yy87; goto yy4; yy4: -#line 625 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 625 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { return 0; } #line 103 "<stdout>" yy5: yyaccept = 0; @@ -489,7 +489,7 @@ yy14: ++YYCURSOR; goto yy15; yy15: -#line 619 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 619 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { /* this is the case where we have less data than planned */ php_error_docref(NULL TSRMLS_CC, E_NOTICE, "Unexpected end of serialized data"); @@ -518,10 +518,10 @@ yy22: ++YYCURSOR; goto yy23; yy23: -#line 507 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 507 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { size_t len, len2, len3, maxlen; - int elements; + long elements; char *class_name; zend_class_entry *ce; zend_class_entry **pce; @@ -659,7 +659,7 @@ yy29: ++YYCURSOR; goto yy30; yy30: -#line 499 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 499 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { INIT_PZVAL(*rval); @@ -691,10 +691,10 @@ yy36: ++YYCURSOR; goto yy37; yy37: -#line 477 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 477 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { - int elements = parse_iv(start + 2); - + long elements = parse_iv(start + 2); + /* use iv() not uiv() in order to check data range */ *p = YYCURSOR; if (elements < 0) { @@ -737,7 +737,7 @@ yy43: ++YYCURSOR; goto yy44; yy44: -#line 449 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 449 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { size_t len, maxlen; char *str; @@ -854,11 +854,11 @@ yy55: ++YYCURSOR; goto yy56; yy56: -#line 442 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 442 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { *p = YYCURSOR; INIT_PZVAL(*rval); - ZVAL_DOUBLE(*rval, zend_strtod(start + 2, NULL)); + ZVAL_DOUBLE(*rval, zend_strtod((const char *)start + 2, NULL)); return 1; } #line 520 "<stdout>" @@ -921,7 +921,7 @@ yy66: ++YYCURSOR; goto yy67; yy67: -#line 427 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 427 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { *p = YYCURSOR; INIT_PZVAL(*rval); @@ -965,7 +965,7 @@ yy73: ++YYCURSOR; goto yy74; yy74: -#line 420 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 420 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { *p = YYCURSOR; INIT_PZVAL(*rval); @@ -983,7 +983,7 @@ yy77: ++YYCURSOR; goto yy78; yy78: -#line 413 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 413 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { *p = YYCURSOR; INIT_PZVAL(*rval); @@ -994,7 +994,7 @@ yy79: ++YYCURSOR; goto yy80; yy80: -#line 406 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 406 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { *p = YYCURSOR; INIT_PZVAL(*rval); @@ -1027,9 +1027,9 @@ yy85: ++YYCURSOR; goto yy86; yy86: -#line 383 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 383 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { - int id; + long id; *p = YYCURSOR; if (!var_hash) return 0; @@ -1076,9 +1076,9 @@ yy91: ++YYCURSOR; goto yy92; yy92: -#line 362 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 362 "/usr/src/php-cvs/ext/standard/var_unserializer.re" { - int id; + long id; *p = YYCURSOR; if (!var_hash) return 0; @@ -1099,7 +1099,7 @@ } #line 761 "<stdout>" } -#line 627 "/usr/src/php/php5/ext/standard/var_unserializer.re" +#line 627 "/usr/src/php-cvs/ext/standard/var_unserializer.re" return 0;
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php