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

"Memcache session handling, zend_mm_heap corrupted" on bugs.php.net #65

Closed
TysonAndre opened this issue Mar 6, 2020 · 2 comments · Fixed by #76
Closed

"Memcache session handling, zend_mm_heap corrupted" on bugs.php.net #65

TysonAndre opened this issue Mar 6, 2020 · 2 comments · Fixed by #76

Comments

@TysonAndre
Copy link
Contributor

TysonAndre commented Mar 6, 2020

https://bugs.php.net/bug.php?id=77900

Sorry if you've already seen this. (The issue has no assignee or response comments on bugs.php.net, so I can't tell, and I personally don't check bugs.php.net frequently or know how to subscribe to bug reports for a given extension.)

The bug was for memcache 3.0.9 with php 7.3. I can't reproduce the zend_mm_heap issue in 4.0.5.2 from PECL (trying with NTS on 7.4 only). However, I see a memory leak (same leak 4 times) instead in 4.0.5.2, when running with a PHP 7.4.3-dev NTS DEBUG build.
(The impact is hopefully limited to just a few http requests for sessions, and should get garbage collected after the HTTP response is sent? I haven't investigated further)

» php --no-php-ini --php-ini test.ini test.php
array(1) {                                     
  ["test"]=>                                   
  bool(true)                                   
}                                              
[Fri Mar  6 15:40:14 2020]  Script:  '/path/to/test.php'
/tmp/pear/temp/memcache/php7/memcache_queue.c(35) :  Freeing 0x00007f0542e7d700 (208 bytes), script=/path/to/test.php
Last leak repeated 3 times
=== Total 4 memory leaks detected ===
This was referenced Oct 8, 2020
@thesource93 thesource93 linked a pull request Oct 16, 2020 that will close this issue
@TysonAndre
Copy link
Contributor Author

Should this and some of the other tickets mentioned in the description of #76 be closed?

added support for reflection 64
added test for 53
fixed failing tests including 46 and 57
fixed underlying issue in 7

@thesource93
Copy link
Contributor

fixed in #76

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 a pull request may close this issue.

2 participants