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 752341 - GDM fails to publish correct X11-display to DBus
GDM fails to publish correct X11-display to DBus
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
3.16.x
Other Linux
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2015-07-13 18:21 UTC by glowfish74
Modified: 2016-11-10 19:00 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
daemon: update X11DisplayName on register display (2.43 KB, patch)
2016-11-10 16:14 UTC, Mathias Reck 		committed Details | Review

Description glowfish74 2015-07-13 18:21:05 UTC 
GDM seems to have lost the functionality to register its display name with DBus since 3.14

GDM on Fedora 22 only delivers empty strings when querying the interface:

qdbus --system org.gnome.DisplayManager /org/gnome/DisplayManager/Displays/3088753552 org.gnome.DisplayManager.Display.GetX11DisplayName

-> [Invalid UTF-8]

This also affects logind that is not able to publish the display name correctly, the "Display" property is always empty.

GDM published its information correctly.

Version tested: GDM 3.16.2
Comment 1 glowfish74 2015-07-13 18:32:29 UTC 
Of course, the second to last sentence should be:

GDM 3.14 published its information correctly.
Comment 2 Michael Catanzaro 2016-01-11 22:05:32 UTC 
Just for ease of reproducing, the equivalent gdbus call:

$ gdbus call --system --dest org.gnome.DisplayManager --object-path /org/gnome/DisplayManager/Displays/94122005586688 --method org.gnome.DisplayManager.Display.GetX11DisplayName

And result (with GDM 3.18.2) is still:

('[Invalid UTF-8]',)
Comment 3 Mathias Reck 2016-11-10 16:14:01 UTC 
Created attachment 339505 [details] [review]
daemon: update X11DisplayName on register display

When a display registered, the sessions 'display-name' was already updated.
The displays 'x11-display-name' however was not, so I've just added that.
Of course that also meant that the 'x11-display-name' could no longer be
constructor only.
Comment 4 Ray Strode [halfline] 2016-11-10 19:00:07 UTC 
Attachment 339505 [details] pushed as a0f63b3 - daemon: update X11DisplayName on register display
Comment 5 Ray Strode [halfline] 2016-11-10 19:00:25 UTC 
(Thanks!)