Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 8.2 - msgpack_check_ht_is_map assertion error in ZEND_HASH_FOREACH_BUCKET #165

Closed
TysonAndre opened this issue Aug 13, 2022 · 0 comments

Comments

@TysonAndre
Copy link

!zend_array_is_list(array) can be used instead - it's internally used by https://php.net/array_is_list for PHP 8.1+

In PHP 8.2, the zend_array implementation was split into 2 implementations: (1) the original (aka MAP), (2) a separate, more memory efficient case (PACKED) for the representation of packed arrays - packed arrays only store the values, not the buckets

php: /msgpack-php/msgpack_pack.c:28: msgpack_check_ht_is_map: Assertion `!(((__ht)->u.flags & (1<<2)) != 0)' failed.                                                                                                               
Aborted (core dumped)                                                                                                     
                                                                                                                       
Termsig=6
@m6w6 m6w6 closed this as completed in 4242098 Oct 7, 2022
m6w6 added a commit that referenced this issue Oct 7, 2022
* Fixed PHP-8.2 compatibility (see gh issue #165)

RC1:
* Fixed PHP-8.1 compatibility (see gh issues #161, #157, and #156)
* Added support for ZEND_ACC_NOT_SERIALIZABLE and magic __{,un}serialize
m6w6 added a commit that referenced this issue Jun 2, 2023
No changes from RC2.
* RC2:
  * Fix PHP-8.2 compatibility (see gh issue #165)
* RC1:
  * Fix PHP-8.1 compatibility (see gh issues #161, #157, and #156)
  * Add support for ZEND_ACC_NOT_SERIALIZABLE and magic __{,un}serialize
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant