3f / flightsdcpp

FlightSDC++ project - client for Direct Connect protocol

	More ...

*update every hour

[ ☕ ]

flightsdcpp's Issues

#8: memory leaks :: SearchFrm

в общем забил я на перенос задачек и wiki от разных репо - даже если bitbucket вконец загнется, то не велика потеря

ранее обнаруженные в 3529510

фишка судя по всему с TypedTreeListViewCtrl::deleteAllItems, возможно дебош совместный с insertItem...

нужно глянуть как можно скорее

Для заметки:

Утечка наступает при рассылки сообщений листу - LVM_DELETEALLITEMS в работе с фильтрацией.

SearchInfoList заполняется после очистки из его парентов, но часть записей может присутствовать только в конечном списке(?), соответственно deleteAllItems() проходиться только по parents, а GetItemCount будет равен 0. Вот и получается что мы не можем сослаться на часть записей для высвобождения из heap по завершению

Основной вопрос, почему часть записей летит мимо парента. Проверить это можно переписав очистку следующим образом:

for(SearchInfoList::ParentMap::const_iterator it = ctrlResults.getParents().begin(); it != ctrlResults.getParents().end(); ++it){
    ctrlResults.deleteItem((*it).second.parent);
}

далее можно тогда:

while(ctrlResults.GetItemCount() > 0){
    SearchInfo* si = ctrlResults.getItemData(0);
    //delete si;
    ctrlResults.DeleteItem(0);
}

но если высвобождать оставшиеся с delete, то иногда может возникать break инструкций, значит где-то когда-то это продолжает использоваться

о_о