Bug 200213

What window decoration are you using? That might be an important information for reproducing

Kubuntu's default, which is Ozone.

cannot reproduce neither in RC2 nor in trunk

*** Bug 202572 has been marked as a duplicate of this bug. ***

*** Bug 211439 has been marked as a duplicate of this bug. ***

Martin told me that it could be an upstream Qt bug:

http://bugreports.qt.nokia.com/browse/QTBUG-6934

SVN commit 1064056 by luebking:

avoid "deep" QVector copy

CCBUG: 217040


 M  +1 -1      geometry.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1064056

KWin does not crash on my system with KDE 4.3.85 and Qt 4.6.0_20091222 anymore.

*** Bug 220775 has been marked as a duplicate of this bug. ***

seems to be fixed

*** Bug 237816 has been marked as a duplicate of this bug. ***

reopening.

actually "foreach" creates an implicitly shared copy despite the return is a
reference... so to bypass this bug we'd have to replace "foreach" by eg. an stl
run

*** Bug 246619 has been marked as a duplicate of this bug. ***

*** Bug 247080 has been marked as a duplicate of this bug. ***

*** Bug 250984 has been marked as a duplicate of this bug. ***

Created attachment 52264 [details]
New crash information added by DrKonqi

Kwin crashed while changing the theme for the window borders.

*** Bug 258964 has been marked as a duplicate of this bug. ***

*** Bug 266300 has been marked as a duplicate of this bug. ***

I am reopening this bug as was probably intended with comment #12. We get this bug with KDE 4.6.0 / Qt 4.7.0, too (see bug 266300), so it is not fixed. The upstream "bug" mentioned in comment #6 is not even confirmed, and I doubt it is a Qt bug.

*** Bug 267839 has been marked as a duplicate of this bug. ***

workspace.cpp:1641 void Workspace::setNumberOfDesktops(int n) is now also called by the desktopgrid effect (and not only through loadDesktopSettings()
so it desperately lacks
    workarea.clear();
    workarea.resize(n + 1);
    restrictedmovearea.clear();
    restrictedmovearea.resize(n + 1);
    oldrestrictedmovearea.clear();
    oldrestrictedmovearea.resize(n + 1);
after 
  desktopCount_ = n;

just like loadDesktopSettings() - or we just move that entirely there....

w/o any testing i'm selfconfident enough to mark this as new - blame immediate music ;-P

*** Bug 268711 has been marked as a duplicate of this bug. ***

Git commit 40b1f456da80ff24223920749128b742280506bb by Thomas Lübking.
Committed on 07/03/2011 at 18:41.
Pushed by luebking into branch 'master'.

fix & cleanup Workspace::setNumberOfDesktops()

encapsulate all relevant (vector resizing) code to
Workspace::setNumberOfDesktops and invoke it from
Workspace::loadDesktopSettigns, this will likely
fix
CCBUG:200213

M  +25   -42   kwin/workspace.cpp     

http://commits.kde.org/kde-workspace/40b1f456da80ff24223920749128b742280506bb

Git commit f3f255fef1be7d36c0a6fb931a46f52178440cb3 by Thomas Lübking.
Committed on 13/04/2011 at 22:58.
Pushed by luebking into branch 'KDE/4.6'.

fix & cleanup Workspace::setNumberOfDesktops()

CCBUG: 200213

M  +24   -36   kwin/workspace.cpp     

http://commits.kde.org/kde-workspace/f3f255fef1be7d36c0a6fb931a46f52178440cb3

(In reply to comment #24)
> Git commit f3f255fef1be7d36c0a6fb931a46f52178440cb3 by Thomas Lübking.
> Committed on 13/04/2011 at 22:58.
> Pushed by luebking into branch 'KDE/4.6'.
> 
> fix & cleanup Workspace::setNumberOfDesktops()
> 
> CCBUG: 200213
> 
> M  +24   -36   kwin/workspace.cpp     
> 
> http://commits.kde.org/kde-workspace/f3f255fef1be7d36c0a6fb931a46f52178440cb3

When building kdebase4-workspace with this patch included (which will be so in 4.6.3) the number of virtual desktop available in the pager applet  (& in kcmshell4 desktop) are reduced to 1 & the  creation of new virtual desktop is not working as expected :
For example if i'm trying to increase the number of virtual desktop from 1 to 6 using kcmshell4 desktop  ,i don't see any creation in the pager applet after.
Maybe the pager applet need some changes too ?

Regards,

Yes, known - but thanks.

The backport isn't "complete" compared to the master version (because i didn't want to change sth. that seemed wrong but harmless) - turned out to cause a functional lack on initial startup.
(if you run the desktopgrid effect and add, then remove a desktop the state will be validated)

The change will be committed later this day. Sorry for trouble.

Git commit 79187a9feb1ae0a0c5f49b7669a65b3606271ea1 by Thomas Lübking.
Committed on 18/04/2011 at 21:16.
Pushed by luebking into branch 'KDE/4.6'.

complete backport for setNumberOfDesktops() fix

CCBUG: 200213
completes backport f3f255fef1be7d36c0a6fb931a46f52178440cb3 to match
master version 40b1f456da80ff24223920749128b742280506bb since the
incomplete backport lead to unset number of desktops on the root window
(applies only after relogin)

M  +6    -9    kwin/workspace.cpp     

http://commits.kde.org/kde-workspace/79187a9feb1ae0a0c5f49b7669a65b3606271ea1

is this one fixed?

"likely" - the CC was intentional since i could not reproduce this crash (compiler aggression thing?!)
The altered code however was prone to invalidate the vector, causing out-of-bound access - it nevertheless could still be a Qt bug - or a combination of both ;-)

For the general bugzilla untrashing, I'll just mark it fixed - and wait for new occurrences... (with new traces)