@@ -22,13 +22,13 @@ typedef struct
2222 void * next ;
2323} var_entries ;
2424
25- #define MSGPACK_UNSERIALIZE_ALLOC_STACK (_unpack ) \
26- if (_unpack->deps <= 0) { \
27- *obj = _unpack->retval; \
28- msgpack_stack_push(_unpack->var_hash, obj, false ); \
29- } else { \
30- ALLOC_INIT_ZVAL(*obj); \
31- msgpack_stack_push(_unpack->var_hash, obj, true); \
25+ #define MSGPACK_UNSERIALIZE_ALLOC_STACK (_unpack ) \
26+ if (_unpack->deps <= 0) { \
27+ *obj = _unpack->retval; \
28+ msgpack_stack_push(_unpack->var_hash, obj, 0 ); \
29+ } else { \
30+ ALLOC_INIT_ZVAL(*obj); \
31+ msgpack_stack_push(_unpack->var_hash, obj, 1); \
3232 }
3333
3434#define MSGPACK_UNSERIALIZE_ALLOC_VALUE (_unpack ) \
@@ -42,9 +42,8 @@ typedef struct
4242
4343#define MSGPACK_UNSERIALIZE_FINISH_ITEM (_unpack , _count ) \
4444 msgpack_stack_pop(_unpack->var_hash, _count); \
45- long deps = _unpack->deps - 1; \
46- _unpack->stack[deps]--; \
47- if (_unpack->stack[deps] == 0) { \
45+ _unpack->stack[_unpack->deps-1]--; \
46+ if (_unpack->stack[_unpack->deps-1] == 0) { \
4847 _unpack->deps--; \
4948 }
5049
@@ -118,7 +117,7 @@ inline static int msgpack_var_access(
118117}
119118
120119inline static void msgpack_stack_push (
121- php_unserialize_data_t * var_hashx , zval * * rval , bool save )
120+ php_unserialize_data_t * var_hashx , zval * * rval , zend_bool save )
122121{
123122 var_entries * var_hash , * prev = NULL ;
124123
@@ -197,7 +196,7 @@ inline static zend_class_entry* msgpack_unserialize_class(
197196 zval * * container , char * class_name , size_t name_len )
198197{
199198 zend_class_entry * ce , * * pce ;
200- bool incomplete_class = false ;
199+ zend_bool incomplete_class = 0 ;
201200 zval * user_func , * retval_ptr , * * args [1 ], * arg_func_name ;
202201 TSRMLS_FETCH ();
203202
@@ -262,7 +261,7 @@ inline static zend_class_entry* msgpack_unserialize_class(
262261 "it was called for" , __FUNCTION__ , class_name );
263262 }
264263
265- incomplete_class = true ;
264+ incomplete_class = 1 ;
266265 ce = PHP_IC_ENTRY ;
267266 }
268267
@@ -299,7 +298,8 @@ void msgpack_unserialize_var_init(php_unserialize_data_t *var_hashx)
299298 var_hashx -> first_dtor = 0 ;
300299}
301300
302- void msgpack_unserialize_var_destroy (php_unserialize_data_t * var_hashx , bool err )
301+ void msgpack_unserialize_var_destroy (
302+ php_unserialize_data_t * var_hashx , zend_bool err )
303303{
304304 void * next ;
305305 long i ;
@@ -529,10 +529,12 @@ int msgpack_unserialize_map(
529529int msgpack_unserialize_map_item (
530530 msgpack_unserialize_data * unpack , zval * * container , zval * key , zval * val )
531531{
532+ long deps ;
532533 TSRMLS_FETCH ();
533534
534535 if (MSGPACK_G (php_only ))
535536 {
537+ zend_class_entry * ce ;
536538 if (Z_TYPE_P (key ) == IS_NULL )
537539 {
538540 unpack -> type = MSGPACK_SERIALIZE_TYPE_NONE ;
@@ -556,7 +558,7 @@ int msgpack_unserialize_map_item(
556558 }
557559 else if (Z_TYPE_P (val ) == IS_STRING )
558560 {
559- zend_class_entry * ce = msgpack_unserialize_class (
561+ ce = msgpack_unserialize_class (
560562 container , Z_STRVAL_P (val ), Z_STRLEN_P (val ));
561563
562564 if (ce == NULL )
@@ -575,7 +577,7 @@ int msgpack_unserialize_map_item(
575577 {
576578 unpack -> type = MSGPACK_SERIALIZE_TYPE_NONE ;
577579
578- zend_class_entry * ce = msgpack_unserialize_class (
580+ ce = msgpack_unserialize_class (
579581 container , Z_STRVAL_P (key ), Z_STRLEN_P (key ));
580582
581583 if (ce == NULL )
@@ -698,7 +700,7 @@ int msgpack_unserialize_map_item(
698700 zval_ptr_dtor (& key );
699701 msgpack_stack_pop (unpack -> var_hash , 2 );
700702
701- long deps = unpack -> deps - 1 ;
703+ deps = unpack -> deps - 1 ;
702704 unpack -> stack [deps ]-- ;
703705 if (unpack -> stack [deps ] == 0 )
704706 {
0 commit comments