Memory leak caused by meminfo_dump about php-meminfo HOT 5 OPEN

Hi there, speaking of that test (tests/dump-memory_leak.phpt), compiling fails that particular test for me on some builds and passes on others, consistently.

• EL6 PHP 7.0: PASS
• EL6 PHP 7.1: FAIL
• EL6 PHP 7.2: FAIL
• EL6 PHP 7.3: PASS
• EL7 PHP 7.0: PASS
• EL7 PHP 7.1: PASS
• EL7 PHP 7.2: PASS
• EL7 PHP 7.3: FAIL

An example fail output from EL6 PHP 7.2:

=====================================================================
TEST RESULT SUMMARY
---------------------------------------------------------------------
Exts skipped    :    0
Exts tested     :   15
---------------------------------------------------------------------

Number of tests :   12                 1
Tests skipped   :   11 ( 91.7%) --------
Tests warned    :    0 (  0.0%) (  0.0%)
Tests failed    :    1 (  8.3%) (100.0%)
Expected fail   :    0 (  0.0%) (  0.0%)
Tests passed    :    0 (  0.0%) (  0.0%)
---------------------------------------------------------------------
Time taken      :    2 seconds
=====================================================================

=====================================================================
FAILED TEST SUMMARY
---------------------------------------------------------------------
meminfo_dump check there's no memory leak [tests/dump-memory_leak.phpt]
=====================================================================

edit: added php 7.3 to results

from php-meminfo.

Thanks @squio for this report,

We have some tests to make sure we don't leak: https://github.com/BitOne/php-meminfo/blob/master/extension/php7/tests/dump-memory_leak.phpt

But it seems it's not sufficient enough.

Do you have the same problem on a more recent version of PHP? Can you try without xdebug enabled? And finally, do you have any code that you can provide so I can do some testing?

Thank you very much!

from php-meminfo.

@jc21 I am having the same problem on CentOS 6.9 running PHP 7.1.11. Tried it on a Docker container based on php:7.1-apache based on Debian 8 and it works, all tests passing. I will try to run some more tests to see if it is CentOS specific.

I was already having this issue from my PHP routines, which led me to try out this repo. The exact same amount of memory is leaking in the test (2097152 bytes), so I am thinking this might be something OS or PHP specific.

from php-meminfo.

I've implemented the builds on a different hardware and now I'm getting different results:

• EL6 PHP 7.0: PASS
• EL6 PHP 7.1: FAIL
• EL6 PHP 7.2: FAIL
• EL6 PHP 7.3: FAIL (different)
• EL7 PHP 7.0: PASS
• EL7 PHP 7.1: PASS
• EL7 PHP 7.2: PASS
• EL7 PHP 7.3: PASS (different)

So my best guess is that this test is defunct in some way ¯_(ツ)_/¯

from php-meminfo.

I managed to build a Docker image that is able to reproduce this issue.

docker pull gusantoniassi/php-meminfo-memory-leak-bug
docker run -it --rm gusantoniassi/php-meminfo-memory-leak-bug bash
cd /opt/php-meminfo/extension/php7
make test
# test dump-memory_leak.phpt should fail

As you can see from the Dockerfile the installation is pretty minimal, so I believe it is either an issue with EL6 or with the remirepo.net's PHP packages.

from php-meminfo.