Key repeats forever if a shell popover steals focus during a keypress

Bug #1964442 reported by Matthias Klose
40
This bug affects 5 people
Affects Status Importance Assigned to Milestone
GNOME Shell
Fix Released
Unknown
Mutter
Fix Released
Unknown
mutter (Ubuntu)
Fix Released
High
Marco Trevisan (Treviño)
Jammy
Fix Released
High
Marco Trevisan (Treviño)

Bug Description

This is a regression not seen until this week (not seen until March 4 at least). Calling debsign from a terminal pops up the window to enter the key, but the terminal is sent continuous <Return> keys and the terminal scrolls down with empty input lines. You can still sign the file, however you end up with an empty scroll buffer if the signing takes too long time.

Matthias Klose (doko)
tags: added: rls-jj-incoming
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report. Is that in a standard Ubuntu session, under x11 or wayland? could you attach the journalctl log from around the time of the problem?

Changed in gnome-terminal (Ubuntu):
importance: Undecided → Low
status: New → Incomplete
tags: added: rls-jj-notfixing
removed: rls-jj-incoming
tags: added: rls-jj-incoming
removed: rls-jj-notfixing
Revision history for this message
Jeremy Bícha (jbicha) wrote :

I experienced a similar issue today. I ran `snap install app` in my terminal. It popped up a PolicyKit login window and my terminal started filling with empty input lines.

Around the same time, I had this in my journal log repeatedly:

gnome-shell: Object 0x5614373df110 of type IBusText has been finalized while it was still owned by gjs, this is due to invalid memory management.

Therefore, I'm reassigning to gnome-shell and ibus. I don't think this issue is specific to gnome-terminal. You might eventually see it with a different terminal.

I am using the default Ubuntu session on Wayland.

affects: gnome-terminal (Ubuntu) → gnome-shell (Ubuntu)
Changed in gnome-shell (Ubuntu):
status: Incomplete → Confirmed
Changed in ibus (Ubuntu):
status: New → Confirmed
tags: added: jammy
summary: - [jammy Regression] signing a changes file in gnome-terminal sends
- <Return> keys to the terminal
+ [jammy][regression] gnome-shell PolicyKit password prompt sends <Return>
+ keys to the terminal
Revision history for this message
Daniel van Vugt (vanvugt) wrote (last edit ): Re: [jammy][regression] gnome-shell PolicyKit password prompt sends <Return> keys to the terminal

The log message in comment #2 is bug 1965673. I'm seeing it in everyone's logs today, including my own. And without using password dialogs. So unassigning ibus here. Use bug 1965673 for that.

no longer affects: ibus (Ubuntu)
Revision history for this message
Alex Murray (alexmurray) wrote :

I personally don't think this should be low priority - this affects any application which ends up causing the gnome shell prompt dialog to appear - so in my case when reading my email and opening a GPG encrypted email I get prompted for my GPG passphrase - whilst this is happening my email client is repeatedly getting <Return> sent to it, causing it to advance forward to the next email and mark the current one as read - so this has a real usability impact IMO. This is not just relevant to terminal applications etc.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

It might be related to https://gitlab.gnome.org/GNOME/mutter/-/issues/2189 but ideally someone should (please) open a new upstream bug to be sure: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues

Changed in gnome-shell (Ubuntu):
importance: Low → Undecided
Revision history for this message
Alex Murray (alexmurray) wrote :
Revision history for this message
Sebastien Bacher (seb128) wrote :

Daniel, could you work on this one?

Changed in gnome-shell (Ubuntu):
importance: Undecided → High
assignee: nobody → Daniel van Vugt (vanvugt)
tags: added: dt-281
Changed in gnome-shell (Ubuntu Jammy):
status: Confirmed → In Progress
Changed in mutter (Ubuntu Jammy):
assignee: nobody → Daniel van Vugt (vanvugt)
status: New → In Progress
importance: Undecided → High
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

The conversation is mostly in https://gitlab.gnome.org/GNOME/mutter/-/issues/2194 now.

summary: - [jammy][regression] gnome-shell PolicyKit password prompt sends <Return>
- keys to the terminal
+ Key repeats forever if a shell popover steals focus during a keypress
Changed in gnome-shell (Ubuntu Jammy):
assignee: Daniel van Vugt (vanvugt) → Marco Trevisan (Treviño) (3v1n0)
Changed in mutter (Ubuntu Jammy):
assignee: Daniel van Vugt (vanvugt) → Marco Trevisan (Treviño) (3v1n0)
Changed in gnome-shell (Ubuntu Jammy):
status: In Progress → Won't Fix
Changed in mutter (Ubuntu Jammy):
status: In Progress → Fix Committed
Changed in gnome-shell (Ubuntu):
status: In Progress → Invalid
Changed in gnome-shell (Ubuntu Jammy):
status: Won't Fix → Invalid
no longer affects: gnome-shell (Ubuntu)
no longer affects: gnome-shell (Ubuntu Jammy)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mutter - 42.0-3ubuntu1

---------------
mutter (42.0-3ubuntu1) jammy; urgency=medium

  [ Marco Trevisan (Treviño) ]
  * debian/patches: Cherry-pick various upstream commits for 42.1:
    + onscreen/native: Fall back if COPY_MODE_SECONDARY_GPU fails to init
      (LP: #1964037, #1959888)
  * debian/patches: Allow any drag timestamp as drag start serial (LP: #1964541)
  * debian/patches: Move x11-fractional scaling under ubuntu namespace
  * debian/patches: Account ClutterStage grabs on Wayland key focus sync
    (LP: #1964442)
  * debian/libmutter-10-0.symbols: Update including new internal symbols
  * debian/patches: Avoid memory errors when comparing gamma values
  * debian/patches: Fix privacy-screen and connectors updates with
    triple-buffering (LP: #1966178)
  * debian/patches: Ensure privacy screen settings are applied on startup
    (LP: #1966178)

  [ Jeremy Bicha ]
  * Merge from Debian unstable. Remaining changes:
    - debian/patches/x11-Add-support-for-fractional-scaling-using-Randr.patch:
      + X11: Add support for fractional scaling using Randr
    - Add triple-buffering patch
    - debian/libmutter-10-0.symbols: Add symbols for triple buffering patch
    - ubuntu/wayland-data-device-Allow-any-drag-timestamp-as-drag-star.patch:
      + handle DnD for old (snapped) gtk apps that used wrong wayland serials

mutter (42.0-3) unstable; urgency=medium

  [ Marco Trevisan (Treviño) ]
  * debian/patches: Cherry-pick upstream fixes targeting 42.1
  * debian/patches: Ensure repick happens on actors visibility changed
    (LP: #1964545)

  [ Jeremy Bicha ]
  * Release to unstable

 -- Marco Trevisan (Treviño) <email address hidden> Wed, 13 Apr 2022 03:30:39 +0200

Changed in mutter (Ubuntu Jammy):
status: Fix Committed → Fix Released
tags: added: fixed-in-42.1 fixed-upstream
Changed in gnome-shell:
status: Unknown → Fix Released
Changed in mutter:
status: Unknown → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.