GNOME Bugzilla – Bug 773187
seg fault when changing typeface
Last modified: 2016-10-18 21:41:06 UTC
Created attachment 337976 [details] patch - remove infinite recursion Disclaimer: This bug is likely a duplicate of #773029 Overview: gucharmap segfaults when I change the typeface via the drop-down select widget. Steps to Reproduce: 1) run gucharmap 2) click on typeface drop-down select widget. 3) observe that list of typefaces appears. 4) select a different typeface. 5) observe gucharmap crashes. Actual Results: gucharmap crashes. Expected Results: I expected the Character Table grid to display the typeface I selected. Build and Hardware: Arch Linux x86_64 Additional Information: This is likely a duplicate of bug 773029. In the event it's not, I didn't want to clutter the bug 773029 with my details. Best I can figure--and I'm fairly inexperienced--the segfault is caused by an infinite loop due to some [glib?] closure invoked repeatedly. I disabled one of the call paths (see attached patch) and the segfault no longer occurs, and gucharmap appears to run correctly. I have no idea if my patch is the proper fix or what that disabled code was intended to do. This is offered as a starting point for further testing.
Created attachment 337977 [details] software version list
Created attachment 337979 [details] gucharmap coredumpctl backtrack
Not reproducible here with steps 1-4, no step 5 (crash) here. I have gtk+ 3.20 while you have 3.22, so possibly it's a gtk+ bug introduced in 3.22. As for patch, gucharmap_charmap_update_text_tags does not call gucharmap_charmap_set_font_desc_internal, so removing the call of the former from the latter wouldn't appear to solve any recursion problem. Indeed a duplicate of bug 773029. *** This bug has been marked as a duplicate of bug 773029 ***
I just noticed my coredump backtrace attachment doesn't contain the details I saw in my gdb backtrace, which better demonstrates the indirect invocation of gucharmap_charmap_set_font_desc_internal via some closure. Shall I submit the gdb backtrace here or on bug 773029?
Here, please.
Created attachment 337983 [details] gucharmap gdb backtrace full
Bah. disregard comment 4. `gucharmap_charmap_set_font_desc_internal` IS NOT called recursively. it kicked off one of the calls to `gucharmap_charmap_update_text_tags` which is why I tested the app with that call removed. Sorry for any mixed signals.