Metacity Not Sending ConfigureNotify events

Bug #1675369 reported by Dave Richards on 2017-03-23
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
metacity (Ubuntu)
Undecided
Unassigned
Xenial
Undecided
Unassigned
Yakkety
Undecided
Unassigned

Bug Description

# Impact
When using seamless applications such as Rdesktop, the window manager needs to send ConfigureNotify events. This is specified in ICCCM section 4.1.5. Without it, software will hang and wait for a response.

This greatly impacts corporate and enterprise users that are trying to integrate Microsoft Windows applications using Rdesktop.

# Proposed Fix
A patch has been created here and should be merged:

https://git.gnome.org/browse/metacity/commit/?h=gnome-3-24&id=f09967fc0f1d65fc7b5057b362b9657154a86079

Patch for 3.18 branch which was used for Xenial upload is here:

https://git.gnome.org/browse/metacity/commit/?h=gnome-3-18&id=64e95c2ec2a1669da2ddab6c29108d718f79dfff

# Test Case
Use rdesktop in seamless mode:
rdesktop "%ProgramFiles%\ThinLinc\WTSTools\seamlessrdpshell.exe" -s "notepad" server_ip

This requires server with windows that is setup for remote access. Also extra software is needed:
https://www.cendio.com/thinlinc/download-register

Opening multiple seamless windows and trying to switch between them should not cause any noticeable delays / hangs.

# Regression Potential
The reporter has verified the fix, and it works. It has also been in Zesty since Mar 20th and so far nobody complained.

description: updated
Changed in metacity (Ubuntu):
status: New → Fix Released
description: updated
Brian Murray (brian-murray) wrote :

Could you write a "Test Case" section that follows the SRU procedure? From https://wiki.ubuntu.com/StableReleaseUpdates.

[Test Case]

 * detailed instructions how to reproduce the bug

 * these should allow someone who is not familiar with the affected
   package to reproduce the bug and verify that the updated package fixes
   the problem.

I updated test case, will that be enough or more detailed instructions are required?

description: updated
Brian Murray (brian-murray) wrote :

No, that's great thanks.

Changed in metacity (Ubuntu Xenial):
status: New → Fix Committed
tags: added: verification-needed

Hello Dave, or anyone else affected,

Accepted metacity into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/metacity/1:3.18.7-0ubuntu0.3 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Dave Richards (drichard) wrote :

I installed this patch on Xenial and it's working. Windows running in seamless mode seem to be properly receiving ConfigureNotify events and are not deadlocking.

tags: added: verification-done
removed: verification-needed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package metacity - 1:3.18.7-0ubuntu0.3

---------------
metacity (1:3.18.7-0ubuntu0.3) xenial; urgency=medium

  * Backport upstream commit to fix send synthetic ConfigureNotify events
    when receiving ConfigureRequests (synthetic_configurenotify_events.diff,
    LP: #1675369).

 -- Dmitry Shachnev <email address hidden> Thu, 23 Mar 2017 21:14:52 +0300

Changed in metacity (Ubuntu Xenial):
status: Fix Committed → Fix Released

The verification of the Stable Release Update for metacity has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Changed in metacity (Ubuntu Yakkety):
status: New → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers