Попробуйте написать внутреннюю функцию в расширении PHP, результаты сообщили об утечке памяти, искали долго, не нашли причину

Недавно я прочитал книгу о расширениях PHP и готов написать функцию, прототип функции выглядит следующим образом:

array_groupBy(string $key, array $input, $forget = false):array

Функция этой функции состоит в том, чтобы сгруппировать $input в соответствии с $key в соответствии со значением группы,$forget указывает, нужно ли удалять массив после группы $key, $clourse является вызываемым типом, и, если существует, значение, соответствующее $key будет передано в замыкание Внутри функции функция замыкания возвращает новую строку в качестве ключа нового массива, например:

введите описание изображения здесь

и вывод:

введите описание изображения здесь

и исходный код

PHP_FUNCTION(array_groupBy){

    zend_string *key;
    zval *input, *val, *key_zval;
    zval group_zval, copy, retval, copy_key_zval;
    zend_bool forget = 0, have_callback = 0;
    HashTable *ht;
    zend_fcall_info fcall_info = empty_fcall_info;
    zend_fcall_info_cache fcall_info_cache = empty_fcall_info_cache;
    int ret;

    ZEND_PARSE_PARAMETERS_START(2, 4)
        Z_PARAM_STR(key)
        Z_PARAM_ARRAY(input)
        Z_PARAM_OPTIONAL
        Z_PARAM_BOOL(forget)
        Z_PARAM_FUNC(fcall_info, fcall_info_cache)
    ZEND_PARSE_PARAMETERS_END();

    if(ZEND_NUM_ARGS() > 3){
        have_callback = 1;
    }

    ht = (HashTable *)emalloc(sizeof(HashTable));
    zend_hash_init(ht, 0, NULL, ZVAL_PTR_DTOR, 0);

    ZEND_HASH_FOREACH_VAL(Z_ARR_P(input), val){

        ZVAL_COPY(&copy, val);

        key_zval = zend_symtable_find(Z_ARR_P(val), key);
        if(have_callback){
            ZVAL_COPY(&copy_key_zval, key_zval);
            fcall_info.retval = &retval;
            fcall_info.params = &copy_key_zval;
            fcall_info.no_separation = 0;
            fcall_info.param_count = 1;
            ret = zend_call_function(&fcall_info, &fcall_info_cache);
            zval_ptr_dtor(&copy_key_zval);
            if(ret != SUCCESS || Z_TYPE(retval) == IS_UNDEF){
                zend_array_destroy(ht);
                RETURN_NULL();
            }
            ZVAL_STR(&copy_key_zval, Z_STR(retval));
        }else{
            ZVAL_STR(&copy_key_zval, zend_string_dup(Z_STR_P(key_zval), 0));
        }
        convert_to_string(&copy_key_zval);
        //GC_REFCOUNT(Z_STR(copy_key_zval)--;

        if(zend_hash_exists(ht, Z_STR(copy_key_zval))){
            group_zval = *zend_hash_find(ht, Z_STR(copy_key_zval));
        }else{
            array_init(&group_zval);
            zend_hash_add_new(ht, Z_STR(copy_key_zval), &group_zval);
        }

        if(forget){
            SEPARATE_ARRAY(&copy);
            zend_symtable_del(Z_ARR(copy), key);
        }
        add_next_index_zval(&group_zval, &copy);

    }ZEND_HASH_FOREACH_END();

    RETURN_ARR(ht);
}

Результат верный, но отчет "обнаружена утечка памяти всего 3", когда я открываю GC_REFCOUNT(Z_STR(copy_key_zval))--, сообщить "Обнаружена 1 утечка памяти". Потому что я не знаю, как отлаживать вышеуказанный код, ведущий меня, чтобы найти долгое время не может найти причину, кто-то может мне помочь?

php_version PHP 7.2.0-dev (cli) (built: Mar 31 2017 10:47:40) ( NTS DEBUG ) Copyright (c) 1997-2017 The PHP Group Zend Engine v3.2.0-dev, Copyright (c) 1998-2017 Zend Technologies

Источник

1 ответ

Решение

Вы не отпускаете строку в copy_key_zval после добавления его в массив. Когда он изначально создан, он будет иметь refcount 1, а когда он будет добавлен в массив, он будет refcount 2; PHP предполагает, что ваша функция висит на нем, пока вы не скажете иначе. Вы должны выпустить его содержание с zval_ptr_dtor() после того, как он был использован.

Аналогично, вы не освобождаете массив в copy после отделения

Источник
Другие вопросы по тегам