Created attachment 110255 [details] le family local go through the validhosts checks xserver returns early in ClientAuthorized > InvalidHost without checking via get_pwnam after the 2566835b commit (Eliminate uninitialized value warnings from access.c) . This is with FamilyLocal , ConvertAddress returns immediatly thus addr is kept null (family is set to 256. Then the commit that eliminate uninitialized value went too far as to check for addr null before siAddrMatch . siAddrMatch called into get_pwname before this commit via siLocalCredAddrMatch then siLocalCredGetId. This breaks gdm initial setup (gnoem-continuous which is at 1.16.99.901 . Tested against master too . The attached patch removed the check for null an addr before siAddrMatch and addrEqual (the latter for consistency as in the same context ).
Fixed in git master by: http://cgit.freedesktop.org/xorg/xserver/commit/?id=9d9bd38fe1454590c303dc936ddac913808bf881 (Sorry, the xserver maintainer doesn't read bugzilla, so only sees patches mailed to xorg-devel mailing list.)
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.