Skip to content

Navigation Menu

Sign in
Sign up

PHP 5.5 compatibility - #2

Open
captolia wants to merge 2 commits into
shevron:master from
captolia:master
Open

PHP 5.5 compatibility #2
captolia wants to merge 2 commits into
shevron:master from
captolia:master

Conversation

@captolia

@captolia captolia commented Apr 28, 2014

Copy link
Copy Markdown

After upgrading to PHP 5.5 I got a 'Bus Error 10' when including the module.

gdb php
(gdb) set args -m
(gdb) run
...
Program received signal EXC_BAD_ACCESS, Could not access memory.
Reason: KERN_PROTECTION_FAILURE at address: 0x0000000100aed4e0
0x00000001001d1409 in _zend_hash_init ()
(gdb) bt
#0 0x00000001001d1409 in _zend_hash_init ()
#1 0x0000000100aea608 in zm_startup_jsonreader (type=<value temporarily unavailable, due to optimizations>, module_number=<value temporarily unavailable, due to optimizations>) at jsonreader.c:776
#2 0x00000001001cb78d in zend_startup_module_ex ()
#3 0x00000001001d273d in zend_hash_apply ()
#4 0x00000001001cbb33 in zend_startup_modules ()
#5 0x0000000100172e47 in php_module_startup ()
#6 0x000000010024df6f in php_cli_startup ()
#7 0x000000010024c4fe in main ()

Allocating the jsonreader_prop_handlers HashTable solved this problem.

Paul Jacobse added 2 commits April 28, 2014 12:58

shevron commented May 16, 2014

Copy link
Copy Markdown
Owner

Thanks. I'm trying to add Travis-CI integration to the project so I can run the tests on different PHP environments. Unfortunately the tests are currently failing (not only on your branch, on all of them). I have very little time to invest in this. Eventually I'll merge things, it may just take a while. If you want to help with getting the tests to run properly on Travis-CI, I'd appreciate any help. I've created a branch captolia/php55-compat which contains some work for that (includes your PR too).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /