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 772005 - Using the text tool with the stylus can crash GIMP
Using the text tool with the stylus can crash GIMP
Status: RESOLVED DUPLICATE of bug 784480
Product: GIMP
Classification: Other
Component: User Interface
git master
Other Linux
: Normal normal
: 2.8
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2016-09-26 19:34 UTC by Elle Stone
Modified: 2018-01-16 20:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gdb messages using text tool - steps of Elle. (3.58 KB, text/plain)
2016-09-26 19:55 UTC, jose americo gobbo
  Details
Enable extension events to get event image coords in inferior_ignore_mode (1.72 KB, patch)
2016-09-27 07:48 UTC, Massimo
rejected Details | Review

Description Elle Stone 2016-09-26 19:34:19 UTC
Another stylus bug, confirmed on Debian testing, gimp commit 98232c2, babl/gegl/gimp updated today. Here are the steps to reproduce the bug:

Open GIMP and change the Preferences so that the default image doesn't show any of the following: menu bar, status bar, scrollbars, and rulers. Make sure only the core pointer and the wacom stylus are at "screen" and everything else is disabled. 

Now close and reopen GIMP. Then use the stylus to make a new image, and select the text tool (still using the stylus). If the device status window is open, the status will probably flick to core pointer and back to status. Then still using the stylus, touch the canvas with the text tool. Then try to change the font size (or color, or etc), still using the stylus. GIMP immediately segfaults.
Comment 1 jose americo gobbo 2016-09-26 19:55:48 UTC
Created attachment 336298 [details]
gdb messages using text tool - steps of Elle.

I confirm in my GIMP Git Master commit: ec64dbb
Ubuntu Gnome 14.04.2
DE GNOME 3.10
Comment 2 Massimo 2016-09-27 07:48:32 UTC
Created attachment 336315 [details] [review]
Enable extension events to get event image coords in inferior_ignore_mode

Making sure this bug was not introduced by my recent stylus related patch
I hacked the attached patch that seems to fix this issue.
Comment 3 Elle Stone 2016-09-28 17:36:25 UTC
The patch seems to have fixed the problem - yeah!
Comment 4 Massimo 2016-09-28 17:58:23 UTC
(In reply to Elle Stone from comment #3)
> The patch seems to have fixed the problem - yeah!

I quickly tested it on Windows, where it is not necessary
the crash is in x11 specific code, and it seems it should
be #ifdeffed out on Windows or the text tool is hard to use
with a tablet on Windows in these conditions.

It is quite easy to end up in a state where hovering the
stylus over the tablet GIMP keeps changing device Stylus->
Core Pointer->Stylus->...

Will test again in the next days comparing with the official
gimp-2.8.18

OTOH on linux it happens also with gimp-2.8
Comment 5 Elle Stone 2016-09-29 19:03:51 UTC
(In reply to Massimo from comment #4)
> It is quite easy to end up in a state where hovering the
> stylus over the tablet GIMP keeps changing device Stylus->
> Core Pointer->Stylus->...
> 

I've noticed this. Also sometimes the stylus will use the tool that was last selected by the mouse, sometimes actually applying the tool (bucket fill, painting a stroke) even though the stylus is the last used input device and is actually hovering over the tablet. But I don't see a clear pattern so it's hard to make a bug report.
Comment 6 Michael Schumacher 2016-10-21 07:27:29 UTC
Do we have any idea why this is influenced by the presence and absence of the UI elements mentions in the initial report?
Comment 7 Michael Schumacher 2016-10-21 07:29:09 UTC
Review of attachment 336315 [details] [review]:

This is something I'd like to have in 2.8.20, doe to be released next week. Do the code changes change any expected behavior?
Comment 8 Massimo 2016-10-25 10:38:03 UTC
Review of attachment 336315 [details] [review]:

>This is something I'd like to have in 2.8.20, doe to be released next week. Do the code changes change any expected behavior?

I don't know and cannot test, but this is a bug in gdk-x11
it should at least warn that the API is used incorrectly
and certainly not crash. Better be fixed there than worked
around in GIMP
Comment 9 Michael Natterer 2018-01-16 20:45:09 UTC
Also one of the obscure tablet-without-ruler crashes.

*** This bug has been marked as a duplicate of bug 784480 ***