graesslin created this revision.
graesslin added reviewers: Plasma on Wayland, KWin.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.

REVISION SUMMARY
  With the NoBorder option set the DecorationBridge won't create
  decorations. Thus we get a nullptr and obviously should not call
  into it. There was already a check for whether decoration is null,
  so that is a rather embarrassing bug.

TEST PLAN
  Test case added which exposes the crash

REPOSITORY
  rKWIN KWin

BRANCH
  dont-crash-no-border-5.7

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

AFFECTED FILES
  autotests/wayland/CMakeLists.txt
  autotests/wayland/dont_crash_no_border.cpp
  shell_client.cpp

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

To: graesslin, #plasma_on_wayland, #kwin
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to