Skip to content

memcached.sess_persistent causes zend_mm_heap corrupted #327

New issue

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

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

Already on GitHub? # to your account

Closed
carlwgeorge opened this issue Feb 15, 2017 · 4 comments
Closed

memcached.sess_persistent causes zend_mm_heap corrupted #327

carlwgeorge opened this issue Feb 15, 2017 · 4 comments
Labels
Milestone

Comments

@carlwgeorge
Copy link

carlwgeorge commented Feb 15, 2017

The IUS project received a bug report against our new php70u-pecl-memcached package for EL7 (PHP 7.0.15, libmemcached 1.0.16). I was able to reproduce it on Fedora 25 (PHP 7.0.15, libmemcached 1.0.18).

Basic reproducer script (thanks @ralfbecker):

<?php
ini_set('session.save_handler', 'memcached');
ini_set('session.save_path', 'localhost:11211');
ini_set('memcached.sess_persistent', true);
session_start();
$_SESSION['test']=true;
session_write_close();
session_start();
var_dump($_SESSION);
session_write_close();
?>
$ php test.php 
zend_mm_heap corrupted
$ gdb --eval-command run --eval-command backtrace --batch --args php test.php 
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
zend_mm_heap corrupted
[Inferior 1 (process 19051) exited with code 01]
No stack.

Removing the ini_set('memcached.sess_persistent', true); line allows the script to execute successfully.

Here are the installed packages.

$ rpm -qa | grep -e php -e memcached | sort
libmemcached-libs-1.0.18-7.fc24.x86_64
memcached-1.4.33-1.fc25.x86_64
php-cli-7.0.15-1.fc25.x86_64
php-common-7.0.15-1.fc25.x86_64
php-json-7.0.15-1.fc25.x86_64
php-pecl-igbinary-2.0.1-1.fc25.x86_64
php-pecl-memcached-3.0.2-1.fc25.x86_64
php-pecl-msgpack-2.0.1-1.fc25.x86_64
@kn007
Copy link

kn007 commented Feb 18, 2017

Same issue here.

laruence added a commit that referenced this issue Feb 19, 2017
@laruence
Copy link
Contributor

should be fixed now...

@kn007
Copy link

kn007 commented Feb 19, 2017

@laruence 谢谢鸟哥~
Thank you! I will try it now.

@kn007
Copy link

kn007 commented Feb 19, 2017

Test 1st Basic reproducer script, it's passed.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants