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

Fixed invalid int -> size_t in params #5

Merged
merged 1 commit into from
Feb 26, 2016

Conversation

willfitch
Copy link
Contributor

This invalid typing causes a segfault on extended stats

@willfitch
Copy link
Contributor Author

@szabolcsbalogh - I'm not sure why the pull request (php/pecl-caching-memcache#4) refers to the NON_BLOCKING_IO_php7, but the php7 branch is much more stable.

This issue fixes a segfault in 034a test.

@willfitch willfitch changed the title Fixed invalid long -> size_t in params Fixed invalid int -> size_t in params Feb 26, 2016
@szabolcsbalogh
Copy link
Contributor

the php7 branch is port of master branch, which is version 2.x.x, and the branch non_blocking_io_php7 is port of non_blocking_io, which is version 3.x.x.

See more about differences in #2

szabolcsbalogh added a commit that referenced this pull request Feb 26, 2016
Fixed invalid int -> size_t in params
@szabolcsbalogh szabolcsbalogh merged commit 88750b4 into websupport-sk:php7 Feb 26, 2016
@willfitch willfitch deleted the php7 branch February 26, 2016 16:50
@willfitch
Copy link
Contributor Author

Ok. I'll look into these warnings and tests for NON_BLOCKING_IO_php7

bof added a commit to bof/pecl-memcache that referenced this pull request Feb 13, 2023
In an attempt to provoke the debug build assertion failure I saw, this
is the first attempt at a test case provoking it.

Doesn't work, yet...

Context on the assertion failure:

  websupport-sk#4  0x00007fc675426c46 in __assert_fail () from /lib64/libc.so.6
  websupport-sk#5  0x00007fc670cbcaf3 in zend_gc_delref (p=0x7fc670bbc000) at /opt/php/include/php/Zend/zend_types.h:1209
  websupport-sk#6  0x00007fc670cbcb59 in zval_delref_p (pz=0x7ffe587973a0) at /opt/php/include/php/Zend/zend_types.h:1245
  websupport-sk#7  0x00007fc670cbcc81 in zval_ptr_dtor_nogc (zval_ptr=0x7ffe587973a0) at /opt/php/include/php/Zend/zend_variables.h:34
  websupport-sk#8  0x00007fc670cbdcbc in ps_read_memcache (mod_data=0x7fc674bed980 <ps_globals+96>, key=0x7fc670b81540, val=0x7ffe58797428,
      maxlifetime=3600) at /srv/dev/yin/phb/build/ourphp-8.1.15-debug/memcache/src/memcache_session.c:385
  websupport-sk#9  0x00007fc673cd7ab3 in php_session_initialize ()
      at /srv/dev/yin/phb/build/ourphp-8.1.15-debug/php-src/ext/session/session.c:444
  websupport-sk#10 0x00007fc673cdafa3 in php_session_start () at /srv/dev/yin/phb/build/ourphp-8.1.15-debug/php-src/ext/session/session.c:1612
  websupport-sk#11 0x00007fc673cded07 in zif_session_start (execute_data=0x7fc670a12d30, return_value=0x7fc670a12d10)
      at /srv/dev/yin/phb/build/ourphp-8.1.15-debug/php-src/ext/session/session.c:2533
  websupport-sk#12 0x00007fc673feea58 in ZEND_DO_ICALL_SPEC_RETVAL_USED_HANDLER ()
      at /srv/dev/yin/phb/build/ourphp-8.1.15-debug/php-src/Zend/zend_vm_execute.h:1297
  websupport-sk#13 0x00007fc674063f3c in execute_ex (ex=0x7fc670a12930)
      at /srv/dev/yin/phb/build/ourphp-8.1.15-debug/php-src/Zend/zend_vm_execute.h:55779
  websupport-sk#14 0x00007fc673f9a4c3 in zend_call_function (fci=0x7ffe58797990, fci_cache=0x7ffe58797970)
      at /srv/dev/yin/phb/build/ourphp-8.1.15-debug/php-src/Zend/zend_execute_API.c:912
  websupport-sk#15 0x00007fc673f9a97d in zend_call_known_function (fn=0x7fc610866500, object=0x0, called_scope=0x0, retval_ptr=0x0,
      param_count=1, params=0x7ffe58797a20, named_params=0x0)
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

Successfully merging this pull request may close these issues.

None yet

2 participants