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

Memory leak on update array #1937

Closed
sergeyklay opened this issue Oct 2, 2019 · 1 comment
Labels
bug

Comments

@sergeyklay
Copy link
Member

@sergeyklay sergeyklay commented Oct 2, 2019

After fix 1f6962d#commitcomment-35331987

I still see memory leak.

manual.php

<?php

use Test\Properties\PropertyUpdate;

$t = new PropertyUpdate();
$t->p1 = [111];
$t->update1();
$ php -d extension=ext/modules/test.so manual.php
[Thu Oct  3 00:17:59 2019]  Script:  '/mnt/work/zephir/php-zephir/manual.php'
/home/klay/.phpenv/versions/7.2.20-zts-debug/include/php/Zend/zend_string.h(134) :  Freeing 0x00007f2309801dc0 (32 bytes), script=/mnt/work/zephir/php-zephir/manual.php
=== Total 1 memory leaks detected ===
$ php -v
PHP 7.2.20 (cli) (built: Jul 14 2019 17:30:04) ( ZTS DEBUG )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.2.20, Copyright (c) 1999-2018, by Zend Technologies

Refs: #1915, #1918

/cc @chrysanthemum

@sergeyklay sergeyklay added the bug label Oct 2, 2019
@chrysanthemum

This comment has been minimized.

Copy link
Contributor

@chrysanthemum chrysanthemum commented Oct 3, 2019

I try to solve~

chrysanthemum added a commit to chrysanthemum/zephir that referenced this issue Oct 11, 2019
@chrysanthemum chrysanthemum referenced this issue Oct 11, 2019
1 of 3 tasks complete
@sergeyklay sergeyklay referenced this issue Oct 11, 2019
1 of 3 tasks complete
sergeyklay added a commit that referenced this issue Oct 11, 2019
Fix #1937 Memory leak on update array
dreamsxin added a commit to dreamsxin/zephir that referenced this issue Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.