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
Closed

Memory leak on update array #1937

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

Comments

@sergeyklay
Copy link
Contributor

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
Copy link
Contributor

chrysanthemum commented Oct 3, 2019

I try to solve~

chrysanthemum added a commit to chrysanthemum/zephir that referenced this issue Oct 11, 2019
sergeyklay added a commit that referenced this issue Oct 11, 2019
dreamsxin pushed 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
Labels
Projects
None yet
Development

No branches or pull requests

2 participants