From 1abff98c0716c9adfe4ee913d8e895f0110f9ab5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thomas=20L=C3=BCbking?= <thomas.luebking@gmail.com>
Date: Fri, 3 May 2013 23:12:53 +0200
Subject: [PATCH] clear m_activeEffects when effects change
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

If a screen with different refreshrate is attached, kwin currently resets
the compositor, altering the effect list.
If at the same time quads are rebuilt, the iteration operates on a pot.
invalid list of dangeling active effects.

Many thanks to Toralf Förster for his testing efforts.

BUG: 308201
FIXED-IN: 4.10.3
REVIEW: 110294
CC: release-team@kde.org
(cherry picked from commit 07d3ac9d8c781755d19c71ccde6d182868a2bfb5)
---
 effects.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/effects.cpp b/effects.cpp
index 073c9f8c9a..f45d81b449 100644
--- a/effects.cpp
+++ b/effects.cpp
@@ -1527,6 +1527,7 @@ void EffectsHandlerImpl::reloadEffect(Effect *effect)
 void EffectsHandlerImpl::effectsChanged()
 {
     loaded_effects.clear();
+    m_activeEffects.clear(); // it's possible to have a reconfigure and a quad rebuild between two paint cycles - bug #308201
 //    kDebug(1212) << "Recreating effects' list:";
     foreach (const EffectPair & effect, effect_order) {
 //        kDebug(1212) << effect.first;