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 598797 - Gnome-Terminal and VTE dump core when using Orca
Gnome-Terminal and VTE dump core when using Orca
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: speech
2.28.x
Other All
: Normal critical
: 2.28.1
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-10-17 21:06 UTC by Steve Holmes
Modified: 2009-10-19 16:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
First debug file when I crashed in terminal; second debug file when restarted (1.88 KB, application/octet-stream)
2009-10-17 21:06 UTC, Steve Holmes
  Details
This is actually the first file I had mentioned. (73.94 KB, application/octet-stream)
2009-10-17 21:26 UTC, Steve Holmes
  Details
fix (5.79 KB, patch)
2009-10-17 23:18 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Steve Holmes 2009-10-17 21:06:27 UTC
Created attachment 145694 [details]
First debug file when I crashed in terminal; second debug file when restarted

If I start up Orca and then go to the terminal Orca still talks fine but as
soon as I press the keypad-enter key, I lose all speech.  If I go to a native
console as root I see still see an Orca process running.  I can kill with 'orca
-q' and then return to my gnome session.  When I attempt to restart Orca from
the gnome run dialog, I only hear "Welcome to Orca." No other speech is heard. 
I have two debug files which I am attaching; orca-debug1.out contains activity
up till when I had to kill Orca from a root terminal; orca-debug2.out contains
all activity from the time I restarted Orca unsuccessfully.
Comment 1 Joanmarie Diggs (IRC: joanie) 2009-10-17 21:21:51 UTC
I cannot reproduce it all of the time, but I was just able to reproduce it in OpenSolaris (build 125). Blek. :-(

I'll take this one. Thanks for the report Steve!
Comment 2 Steve Holmes 2009-10-17 21:26:52 UTC
Created attachment 145696 [details]
This is actually the first file I had mentioned.
Comment 3 Joanmarie Diggs (IRC: joanie) 2009-10-17 22:31:45 UTC
This is a VTE bug. For some reason, calling text.getText(0, -1) is triggering an assertion:

Vte:ERROR:vteaccess.c:946:vte_terminal_accessible_get_text: assertion failed: ((start_offset >= 0) && (end_offset >= -1))

Which causes VTE (or gnome-terminal) to core dump. :-( We can work around it with a text.getText(0, text.characterCount) in the gnome-terminal script. But first I want to know why the assertion is getting triggered because the start_offset is (should in theory be) 0 and the end_offset is (should in theory be) -1.
Comment 4 Joanmarie Diggs (IRC: joanie) 2009-10-17 23:18:40 UTC
Created attachment 145701 [details] [review]
fix

This fixes it for me. Pylints to a 10. And is a simple enough fix. Therefore, it has been committed to master; it has not yet been committed to the gnome-2-28 branch.

Steve: Please pull the latest Orca from git master and see if this solves the problem for you.

Will: Please review. If there is anything that needs changing, I'll do that and then commit to the gnome-2-28 branch.

*NOW* I'll see if I can work out what the VTE guys broke....
Comment 5 Joanmarie Diggs (IRC: joanie) 2009-10-17 23:59:40 UTC
Turns out the VTE guys did something silly. They added a check to see if start_offset > end_offset and, if so, they assume the offsets are reversed and swap 'em. Problem solved by adding an additional check to be sure end_offset ain't -1. I opened bug 598814 against VTE and attached a patch which does that.
Comment 6 Steve Holmes 2009-10-19 11:51:49 UTC
I just tried it out and hitting the KP_Enter key over and over never causes the crash.  It seems to work for me.  not sure about other instances where I thought it failed before but all seems good now.
Comment 7 Willie Walker 2009-10-19 15:18:42 UTC
Review of attachment 145701 [details] [review]:

Looks good.  Commit for 2.28.1.
Comment 8 Joanmarie Diggs (IRC: joanie) 2009-10-19 16:03:58 UTC
Comment on attachment 145701 [details] [review]
fix

Already committed to master; just now committed to the gnome-2-28 branch.