Hello community,

here is the log from the commit of package plasma5-workspace for 
openSUSE:Factory checked in at 2017-07-17 10:20:53
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/plasma5-workspace (Old)
 and      /work/SRC/openSUSE:Factory/.plasma5-workspace.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "plasma5-workspace"

Mon Jul 17 10:20:53 2017 rev:70 rq:509835 version:5.10.3

Changes:
--------
--- /work/SRC/openSUSE:Factory/plasma5-workspace/plasma5-workspace.changes      
2017-07-11 08:23:30.655202891 +0200
+++ /work/SRC/openSUSE:Factory/.plasma5-workspace.new/plasma5-workspace.changes 
2017-07-17 10:20:54.530370845 +0200
@@ -1,0 +2,7 @@
+Wed Jul 12 16:36:16 UTC 2017 - wba...@tmo.at
+
+- Add Notifications-Check-for-corona-to-avoid-crash.patch to
+  prevent a Plasma crash when removing a panel that has a system
+  tray with the Notifications applet (boo#1046500, kde#378508)
+
+-------------------------------------------------------------------

New:
----
  Notifications-Check-for-corona-to-avoid-crash.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ plasma5-workspace.spec ++++++
--- /var/tmp/diff_new_pack.xEfQ98/_old  2017-07-17 10:20:55.538228771 +0200
+++ /var/tmp/diff_new_pack.xEfQ98/_new  2017-07-17 10:20:55.542228208 +0200
@@ -37,6 +37,7 @@
 # PATCH-FIX-OPENSUSE 0001-Ignore-default-sddm-face-icons.patch boo#1001364 
fab...@ritter-vogt.de -- Ignore default sddm face icons
 Patch1:         0001-Ignore-default-sddm-face-icons.patch
 # PATCHES 100-200 and above are from upstream 5.10 branch
+Patch100:       Notifications-Check-for-corona-to-avoid-crash.patch
 # PATCHES 201-300 and above are from upstream master/5.11 branch
 Patch201:       applauncher-allow-to-show-apps-by-name.patch
 Patch202:       logoutdialog-honor-Offer-shutdown-options.patch
@@ -211,6 +212,7 @@
 %setup -q -n plasma-workspace-%{version}
 %patch0 -p1
 %patch1 -p1
+%patch100 -p1
 %patch201 -p1
 %patch202 -p1
 

++++++ Notifications-Check-for-corona-to-avoid-crash.patch ++++++
>From 8a05294e5b3ef1df86f099edde837b8c8d28ccaf Mon Sep 17 00:00:00 2001
From: Kai Uwe Broulik <k...@privat.broulik.de>
Date: Wed, 12 Jul 2017 18:06:28 +0200
Subject: [Notifications] Check for corona to avoid crash

My previous attempt only fixed the case when notifications were directly in a 
panel.
Now it also no longer crashes when removing a panel that has a System Tray with 
the
notifications applet in it.

BUG: 378508
FIXED-IN: 5.8.8

Differential Revision: https://phabricator.kde.org/D6653
---
 applets/notifications/lib/notificationsapplet.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/applets/notifications/lib/notificationsapplet.cpp 
b/applets/notifications/lib/notificationsapplet.cpp
index 4474c8c..f49071c 100644
--- a/applets/notifications/lib/notificationsapplet.cpp
+++ b/applets/notifications/lib/notificationsapplet.cpp
@@ -63,7 +63,7 @@ void NotificationsApplet::onScreenChanges()
 {
     // when removing the panel the applet is in, the containment is being 
destroyed but its corona is still
     // there, rightfully emitting availableScreenRectChanged and then we blow 
up if we try to access it.
-    if (!containment()) {
+    if (!containment() || !containment()->corona()) {
         return;
     }
 
-- 
cgit v0.11.2


Reply via email to