D19634: Fixes minor memory leaks

David Hallas noreply at phabricator.kde.org
Sat Mar 9 15:46:36 GMT 2019


hallas created this revision.
hallas added a reviewer: Konsole.
Herald added a project: Konsole.
Herald added a subscriber: konsole-devel.
hallas requested review of this revision.

REVISION SUMMARY
  Fixes a few minor memory leaks found with address sanitizer.
  The newTabButton and closeTabButton was not passed a parent and is not
  deleted in the destructor of TabbedViewContainer which will cause them
  to be leaked when closing a tab. Instead pass the TabbedViewContainer as
  the parent.
  The same goes for the profileMenu, it was not passed a parent pointer at
  contruction and the setMenu function doesn't take ownership of it, so
  therefore it was also leaked.

TEST PLAN
  Compile konsole with address sanitizer.
  Open and close konsole.

REPOSITORY
  R319 Konsole

BRANCH
  fix_minor_memory_leaks (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D19634

AFFECTED FILES
  src/ViewContainer.cpp

To: hallas, #konsole
Cc: konsole-devel, gennad, maciejn, thsurrel, ngraham, maximilianocuria, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20190309/07ffe956/attachment.html>


More information about the konsole-devel mailing list