MessagePackUnpacker was incorrectly using MessagePack's free_obj php_msgpack_base_free instead of php_msgpack_unpacker_free

Sean-Der · Sean-Der · commit 5b8fac8c751c · 2015-07-02T05:23:01.000Z
diff --git a/msgpack_class.c b/msgpack_class.c
@@ -511,7 +511,7 @@ void msgpack_init_class() /* {{{ */ {
     msgpack_unpacker_ce->create_object = php_msgpack_unpacker_new;
     memcpy(&msgpack_unpacker_handlers, zend_get_std_object_handlers(),sizeof msgpack_unpacker_handlers);
     msgpack_unpacker_handlers.offset = XtOffsetOf(php_msgpack_unpacker_t, object);
-    msgpack_handlers.free_obj = php_msgpack_base_free;
+    msgpack_handlers.free_obj = php_msgpack_unpacker_free;
 
 }
 /* }}} */

Original file line number	Diff line number	Diff line change
`@@ -511,7 +511,7 @@ void msgpack_init_class() /* {{{ */ {`
`511`	`511`	`msgpack_unpacker_ce->create_object = php_msgpack_unpacker_new;`
`512`	`512`	`memcpy(&msgpack_unpacker_handlers, zend_get_std_object_handlers(),sizeof msgpack_unpacker_handlers);`
`513`	`513`	`msgpack_unpacker_handlers.offset = XtOffsetOf(php_msgpack_unpacker_t, object);`
`514`		`- msgpack_handlers.free_obj = php_msgpack_base_free;`
	`514`	`+ msgpack_handlers.free_obj = php_msgpack_unpacker_free;`
`515`	`515`
`516`	`516`	`}`
`517`	`517`	`/* }}} */`