After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 687217 - gnome-shell crash on opening overview in st_widget_get_theme_node
gnome-shell crash on opening overview in st_widget_get_theme_node
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: st
3.6.x
Other Linux
: High critical
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2012-10-30 16:46 UTC by Joe Barnett
Modified: 2012-11-11 21:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gdb session (84.31 KB, application/octet-stream)
2012-10-30 16:46 UTC, Joe Barnett
  Details
updated session w/ more stack (31.47 KB, text/plain)
2012-10-30 19:30 UTC, Joe Barnett
  Details
Patch against 3.6.0 version of NativeWindowPlacement extension. Fixes crash after screen lock. (1.22 KB, patch)
2012-11-11 20:57 UTC, wepmaschda
none Details | Review
Tidy-up patch - prototype (4.79 KB, patch)
2012-11-11 21:09 UTC, wepmaschda
none Details | Review

Description Joe Barnett 2012-10-30 16:46:01 UTC
Created attachment 227655 [details]
gdb session

With gnome shell 3.6.1 on ubuntu 12.10, i'm getting rather frequent crashes when opening the overview (via hotcorner or via super key).  Seems to have started since switching from lightdm to gdm 3.6.1, but not sure if that's the real cause.  gdb session attached
Comment 1 Giovanni Campagna 2012-10-30 16:52:20 UTC
Thanks for taking the time to report this bug.
Unfortunately, that stack trace is missing some elements that will help a lot to solve the problem, so it will be hard for the developers to fix that crash. Can you get us a stack trace with debugging symbols? Please see http://live.gnome.org/GettingTraces for more information on how to do so.
Thanks in advance!

In particular, it seems that you tried to call gjs_dumpstack(), but the result of that is not in the gdb session, it's in gnome-shell stderr.
Comment 2 Joe Barnett 2012-10-30 19:30:52 UTC
Created attachment 227674 [details]
updated session w/ more stack
Comment 3 Joe Barnett 2012-10-30 19:32:18 UTC
ok, got the dumpstack() + more debug packages installed.  looks possibly related to native-window-placement extension, will disable that and see if the crash stops.  I think I can now say it definitely reproduces with lock -> unlock -> overview, as well.
Comment 4 wepmaschda 2012-11-11 20:57:21 UTC
Created attachment 228731 [details] [review]
Patch against 3.6.0 version of NativeWindowPlacement extension. Fixes crash after screen lock.

This is a patch against the 3.6.0 version of the NativeWindowPlacement extension. The solution was mentioned here: https://extensions.gnome.org/extension/18/native-window-placement/
Comment 5 Giovanni Campagna 2012-11-11 21:02:01 UTC
I'm happy to tell you that someone else mentioned this fix by e-mail, and thus I already pushed a fix in the repository and at extensions.gnome.org.

This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.
Comment 6 wepmaschda 2012-11-11 21:09:19 UTC
Created attachment 228733 [details] [review]
Tidy-up patch - prototype

This is first prototype of a patch preparing a tidy-up of the current injection code. This will reduce the interface and conflicts to a minimum.

Can someone look at the code and find out why activating the line marked by "TODO" in _realPositionWindows() or completely removing the function _realPositionWindows() doesn't work? (Haven't had time to take a detailed look at the error messages...)