maintainer-feedback requested: [Bug 220668] x11/xinit

[bugzilla-noreply at freebsd.org]

Duane <parakleta at darkreality.org> has reassigned Bugzilla Automation
<bugzilla at FreeBSD.org>'s request for maintainer-feedback to x11 at FreeBSD.org:
Bug 220668: x11/xinit
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=220668



--- Description ---
`/usr/local/bin/startx` does not correctly calculate the `$displayname` value
used by `xauth`.  Specifically the `xauth` man page states that same-machine
connections should use the form `$hostname/unix:$display`.

The following patch should fix the behaviour.

@@ -192,9 +193,9 @@
     # now add the same credentials to the client authority file
     # if '$displayname' already exists do not overwrite it as another
     # server man need it. Add them to the '$xserverauthfile' instead.
-    for displayname in $authdisplay $hostname$authdisplay; do
+    for displayname in $authdisplay $hostname/unix$authdisplay; do
	 authcookie=`xauth list "$displayname" \
-	 | sed -n "s/.*$displayname[[:space:]*].*[[:space:]*]//p"` 2>/dev/null;
+	     | sed -n "s|.*$displayname[[:space:]*].*[[:space:]*]||p"`
2>/dev/null;
	 if [ "z${authcookie}" = "z" ] ; then
	     xauth -q << EOF 
 add $displayname . $mcookie