Bug: утечка памяти при удалении ReportEngine »

[DrGluck]

Сценарий прежний: создаётся класс, который создаёт экземпляр ReportEngine и модель. Модель может быть owned или нет, всё равно. Вызываем превью, закрываем, удаляем ReportEngine. После этого регулярно не очищается некоторое количество памяти. На тестовом отчёте в 200 страниц утекает по несколько мегабайт. После пары сотен таких действий получилась картинка (см. аттач).

[вложение удалено администратором]

[Arin Alex]

Спасибо. Очень полезная информация. Обязательно гляну.   

[Arin Alex]

А что за утилита на скриншоте ?

[DrGluck]

Process Hacker, но это не принципиально, можно взять Process Explorer от Руссиновича.

[DrGluck]

Ну, собственно, вот:

Code: [Select]

for ( int i = 0; i < 10; i++)
{
    _limeInstance()->previewReport();
}После previewReport память вообще не очищается, за 20 вызовов доходит до 1ГБ.

[вложение удалено администратором]

[Arin Alex]

Проблема несвоевременного удаления объектов в previewReport(), имеет место быть и исправляется через изменение деструктора PreviewReportWindow

PreviewReportWindow::~PreviewReportWindow()
{
    if (m_ownedSettings)
        delete m_settings;
    if (m_previewPage)
        delete m_previewPage;
    delete ui;
}

Дальнейшие тесты выдают (см. аттач.)
Тест проводился на следующем коде:

for (int i=0;i<30;++i){
    LimeReport::ReportEngine* re = new LimeReport::ReportEngine();
   
    QSqlQueryModel* customersModel = new QSqlQueryModel();
    customersModel->setQuery("select * from customers", m_db);
    re->dataManager()->addModel("external_customers_data",customersModel,true);
    QSqlQueryModel* ordersModel = new QSqlQueryModel();
    ordersModel->setQuery("Select * from orders",m_db);
    re->dataManager()->addModel("external_orders_data",ordersModel,true);
   
    re->loadFromFile(fileName);
    re->previewReport();
    delete re;
}
Роста потребления не наблюдается.

[вложение удалено администратором]

[DrGluck]

Сейчас попробуем. Надеюсь в новой версии это уже будет исправлено в самой библиотеке.
Но что делать с падением при попытке удаления открытого отчёта?

[Arin Alex]

Это дело мы тоже проверим, возможно, это как раз связано с выше приведенной ошибкой. 

[DrGluck]

И, этсамое, а зачем проверять m_settings и m_previewPag? delete и так прекрасно справляется.
Я понимаю, что это привычка, но уж лучше тогда завести привычку сразу писать
delete _settings;
_settings = nullptr;
даже если кажется, что это не имеет смысла. А то были случаи, когда потом добавился вызов какой-нибудь функции, которая юзала указатель.

[DrGluck]

Подтверждаю, не течёт. Во-всяком случае в тестовой программе. Сейчас попробую боевую, там отчёт по 300-400 метров жрёт.

[DrGluck]

Дополнение: пустой отчёт, модель не подключена, добавляем Report Header с текстом и запускаем превью 100-200 раз.
У меня память приложения постепенно увеличивается примерно на 5-6 метров. Это, вроде не много, но на отчёте в 500 мегабайт столько утекает иногда на одном запуске.
Проверялось методом "создал репорт, вызвал превью, дождался закрытия, удалил репорт, го то 1".