[Differential] [Request, 42 lines] D1714: [server] Don't destroy SlideInterface when parent SurfaceInterface is destroyed

[graesslin (Martin Gräßlin)]

graesslin created this revision.
graesslin added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Destroying the SlideInterface on the server side before the client has a
  chance to cleanup results in a protocol error:
  wl_display at 1: error 0: invalid object 7
  
  Which would terminate the client. If we would not destroy the resource,
  but only delete the SlideInterface it could result in heap-use-after-free.
  
  So just don't do anything, the client needs to cleanup which will result
  in the SlideInterface being deleted.

REPOSITORY
  rKWAYLAND KWayland

BRANCH
  slide-destroy-surface

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

AFFECTED FILES
  autotests/client/test_wayland_slide.cpp
  src/server/slide_interface.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, Plasma
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160530/06b307ec/attachment.html>