]> cloud.milkyroute.net Git - dolphin.git/commit
Fixes memory leak in KItemListViewAccessible
authorDavid Hallas <david@davidhallas.dk>
Wed, 18 Jul 2018 21:20:18 +0000 (23:20 +0200)
committerElvis Angelaccio <elvis.angelaccio@kde.org>
Wed, 18 Jul 2018 21:20:19 +0000 (23:20 +0200)
commit4536e25ac929d8d37ffbbba0642de662bcc9f891
tree93d852e488e6c5f7f522a263021204b28318d677
parent43ab5bdc5f8b709bc278611a5f6924d6b0aaf1cd
Fixes memory leak in KItemListViewAccessible

Summary: The KItemListViewAccessible class has a list of QAccessibleInterface pointers in a member variable m_cells. The problem is that when new entries are created, the newly allocated pointer is not stored in the list, only a nullptr is store, this renders the cleanup code in the destructor useless. This patch simply stores the pointer in the list, causing the destructor to correctly free the memory.

Test Plan: I found this issue using address sanitizer. Simply building Dolphin with -fsanitize=address and opening a window caused the memory leak.

Reviewers: #dolphin, jtamate, elvisangelaccio

Reviewed By: #dolphin, jtamate, elvisangelaccio

Subscribers: elvisangelaccio, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D14168
src/kitemviews/kitemlistviewaccessible.cpp