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 686186 - slave: add host authorizations individually
slave: add host authorizations individually
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2012-10-15 22:22 UTC by Ray Strode [halfline]
Modified: 2012-10-15 22:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
slave: add host authorizations individually (5.84 KB, patch)
2012-10-15 22:22 UTC, Ray Strode [halfline]
committed Details | Review

Description Ray Strode [halfline] 2012-10-15 22:22:43 UTC
XAddHosts fails if any of the entries in the list fails.
The gdm-initial-setup user isn't available on most systems,
so XAddHosts fails on most systems.

Upshot is, since commit 189cd818bf1592b5b8607793575bcf4e71ba8008
most people have been relying on xauth cookies for X authorization.
xauth cookies aren't nearly as robust across hostname changes, so
that's not ideal.

This commit calls XAddHost() 3 times instead of one XAddHosts
call to work around the problem.  Ideally, we'd only add
gdm-initial-setup when we need it instead of all the time.

That will have to come later.
Comment 1 Ray Strode [halfline] 2012-10-15 22:22:45 UTC
Created attachment 226516 [details] [review]
slave: add host authorizations individually
Comment 2 Ray Strode [halfline] 2012-10-15 22:22:58 UTC
Attachment 226516 [details] pushed as 2bd38e8 - slave: add host authorizations individually