<div>graesslin created this revision.<br />
graesslin added a reviewer: Plasma.<br />
Restricted Application added a project: Plasma.<br />
Restricted Application added a subscriber: plasma-devel.</div><br /><div><strong>REVISION SUMMARY</strong><div><p>Destroying the ServerSideDecorationInterface on the server side before<br />
the client has a chance to cleanup results in a protocol error:<br />
wl_display@1: error 0: invalid object 7</p>

<p>Which would terminate the client. If we would not destroy the resource,<br />
but only delete the ServerSideDecorationInterface it could result in<br />
heap-use-after-free.</p>

<p>So just don't do anything, the client needs to cleanup which will result<br />
in the ServerSideDecorationInterface being deleted.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWAYLAND KWayland</div></div></div><br /><div><strong>BRANCH</strong><div><div>ssd-destroy-surface</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1710" rel="noreferrer">https://phabricator.kde.org/D1710</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/client/test_server_side_decoration.cpp<br />
src/server/server_decoration_interface.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>graesslin, Plasma<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>