D28667: [platforms/x11] Fix valigrind warning copying window icon for windowed mode

David Edmundson noreply at phabricator.kde.org
Tue Apr 7 22:36:00 BST 2020


davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidedmundson requested review of this revision.

REVISION SUMMARY
  windowIcon.pixmap().bits() creates and deletes the QPixmap object.
  
  Which means data is dangling when we call m_winInfo->setIcon()
  
  Surprisingly harmless in real life.

TEST PLAN
  Reran kwin_wayland on X11 with valigrind

REPOSITORY
  R108 KWin

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

AFFECTED FILES
  plugins/platforms/x11/windowed/x11windowed_output.cpp

To: davidedmundson, #kwin
Cc: kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20200407/f568e9b1/attachment.html>


More information about the kwin mailing list