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 690965 - shell crashes when requesting tray while displaying message
shell crashes when requesting tray while displaying message
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: message-tray
3.6.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-01-02 00:41 UTC by Otto Allmendinger
Modified: 2013-01-02 17:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
error log (2.40 KB, text/plain)
2013-01-02 00:42 UTC, Otto Allmendinger
  Details
main: Don't mess up the modal stack when the focus actor is destroyed (982 bytes, patch)
2013-01-02 14:24 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review
main: Rename curFocus to prevFocus (3.07 KB, patch)
2013-01-02 14:24 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review
main: Add a better comment about shifting the modal stack (1.52 KB, patch)
2013-01-02 14:24 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review

Description Otto Allmendinger 2013-01-02 00:41:17 UTC
Requesting the tray while a message is being displayed, then discarding the message by clicking the balloon in the message tray causes gnome-shell to crash.

Steps to reproduce: 

1. run "notify-send foo" in a shell
2. hit Win-M to display tray *during display of message*
3. wait for tray to appear
4. click notification icon
5. click message bubble
6. gnome-shell crashes (alt-tab behaves erroneously, "alt-f2 r" doesn't work..)


Waiting for the message to disappear and opening the tray after that does not seem to produce the bug.


related bugs

https://bugzilla.gnome.org/show_bug.cgi?id=682431
https://bugs.archlinux.org/task/32410
https://bugzilla.redhat.com/show_bug.cgi?id=868261

error log is attached
Comment 1 Otto Allmendinger 2013-01-02 00:42:46 UTC
Created attachment 232486 [details]
error log
Comment 2 Otto Allmendinger 2013-01-02 00:44:52 UTC
the bug I meant to link to is bug 682433
Comment 3 Jasper St. Pierre (not reading bugmail) 2013-01-02 14:24:10 UTC
Created attachment 232526 [details] [review]
main: Don't mess up the modal stack when the focus actor is destroyed

This seems to be an incorrect conversion when we moved from an array
to an object of keys in 3a6b4f3.
Comment 4 Jasper St. Pierre (not reading bugmail) 2013-01-02 14:24:18 UTC
Created attachment 232527 [details] [review]
main: Rename curFocus to prevFocus

This better describes what we're tracking here: the previous
keyboard focus before pushing the modal.
Comment 5 Jasper St. Pierre (not reading bugmail) 2013-01-02 14:24:34 UTC
Created attachment 232528 [details] [review]
main: Add a better comment about shifting the modal stack

It took me a few minutes to realize why, so let's just add this
in for future reference.


Thanks for the extremely clear reproducer! I found the bug.
Comment 6 Giovanni Campagna 2013-01-02 17:29:32 UTC
Review of attachment 232526 [details] [review]:

Yes.
Comment 7 Giovanni Campagna 2013-01-02 17:30:12 UTC
Review of attachment 232527 [details] [review]:

Ok
Comment 8 Giovanni Campagna 2013-01-02 17:31:25 UTC
Review of attachment 232528 [details] [review]:

Ok.
Comment 9 Jasper St. Pierre (not reading bugmail) 2013-01-02 17:33:50 UTC
Attachment 232526 [details] pushed as 55aa0cf - main: Don't mess up the modal stack when the focus actor is destroyed
Attachment 232527 [details] pushed as 2d9ddd4 - main: Rename curFocus to prevFocus
Attachment 232528 [details] pushed as 73388f3 - main: Add a better comment about shifting the modal stack