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 684434 - Segmentation fault when exiting GTK applications in GTK 3.5.18 with an AT client registered.
Segmentation fault when exiting GTK applications in GTK 3.5.18 with an AT cli...
Status: RESOLVED FIXED
Product: at-spi
Classification: Platform
Component: at-spi2-atk
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Li Yuan
At-spi maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-09-20 08:24 UTC by Luke Yelavich
Modified: 2012-09-20 19:13 UTC
See Also:
GNOME target: 3.6
GNOME version: ---

Attachments
Threaded stack trace. (5.25 KB, text/plain)
2012-09-20 08:29 UTC, Luke Yelavich 		Details

Description Luke Yelavich 2012-09-20 08:24:10 UTC 
Reproduceable with gedit, GTK 3.5.18, I get the following trace:

+ Trace 230879

Thread 1 (Thread 0x7f5d78ba1980 (LWP 5732))

  • #0 atk_bridge_adaptor_cleanup
    at bridge.c line 972
  • #1 _gtk_accessibility_shutdown
    at /build/buildd/gtk+3.0-3.5.18/./gtk/a11y/gail.c line 809
  • #2 gtk_main
    at /build/buildd/gtk+3.0-3.5.18/./gtk/gtkmain.c line 1184
  • #3 gedit_main
    at gedit.c line 205
  • #4 main
    at gedit.c line 331 

I believe commit 5debed5ae247c1dd440cb71d0a6d328df74b0844 introduced the problem, as reverting this commit causes the problem to go away.
Comment 1 Luke Yelavich 2012-09-20 08:29:35 UTC 
Created attachment 224813 [details]
Threaded stack trace.

Attached is a threaded stack trace.
Comment 2 Matthias Clasen 2012-09-20 12:33:33 UTC 
This is a bug in atk_bridge_adaptor_cleanup.

Look closely at this code:

  for (ls = clients; ls; ls = ls->next)
    g_free (l->data);
  g_slist_free (clients);
  clients = NULL;


Took me a while to see it too: the list is iterated with ls, but l->data is freed
Comment 3 Mike Gorse 2012-09-20 19:13:33 UTC 
Thanks for tracking this down. Pushed as commit b3210d.