Age | Commit message (Collapse) | Author |
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Originally fixed for Solaris in May 2000 by Steve Swales under
Sun bug 4256527 "xhost messages not internationalized"
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Originally fixed for Solaris in Oct. 2002 under Sun bug 4759889
"xhost does not accept raw IPv6 numeric address"
Before this fix:
% xhost +2001:DB8::11
xhost: unknown address family "2001"
xhost: bad hostname "2001:DB8::11"
After this fix:
% xhost +2001:DB8::11
2001:DB8::11 being added to access control list
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
While parsing Xorg responses it is possible to trigger an out of
boundary read if the response does not contain enough bytes.
In case of IPv4, the padding normally prevents this, but IPv6
addresses can trigger an out of boundary read.
It takes a hostile xorg-server to reproduce this issue. If
os/access.c is adjusted to always use a length of 1, it is possible
to reproduce it and make it visible with an ASAN-compiled xhost.
Reading past the memory boundary could reveal sensitive information
to external DNS servers, because a lookup will be performed.
Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Reviewed-by: Matthieu Herrb <matthieu@herrb.eu>
|
|
Signed-off-by: Mihail Konev <k.mvc@ya.ru>
|
|
Place quotes around the $srcdir, $ORIGDIR and $0 variables to prevent
fall-outs, when they contain space.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
implicit-function-declaration warnings
Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: David Macek <david.macek.0@gmail.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Required on Solaris to expose definitions in system headers that
are not defined in the XPG standards now that xtrans 1.3 defines
_XOPEN_SOURCE to 600 on Solaris.
Fixes build failures:
In file included from /usr/include/rpc/rpcb_clnt.h:34:0,
from /usr/include/rpc/rpc.h:49,
from xhost.c:98:
/usr/include/rpc/rpcb_prot.h:159:3: error: unknown type name ‘u_int’
/usr/include/rpc/rpcb_prot.h:195:3: error: unknown type name ‘u_int’
/usr/include/rpc/rpcb_prot.h:223:2: error: unknown type name ‘u_int’
/usr/include/rpc/rpcb_prot.h:476:1: error: unknown type name ‘u_int’
/usr/include/rpc/rpcb_prot.h:489:1: error: unknown type name ‘u_int’
xhost.c: In function ‘change_host’:
xhost.c:554:11: error: ‘INET6_ADDRSTRLEN’ undeclared (first use in this function)
xhost.c:554:11: note: each undeclared identifier is reported only once for each function it appears in
xhost.c: In function ‘get_hostname’:
xhost.c:665:23: error: ‘NI_MAXHOST’ undeclared (first use in this function)
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
|
|
See http://people.gnome.org/~walters/docs/build-api.txt
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
xhost.c: In function ‘change_host’:
xhost.c:452:13: warning: assignment discards ‘const’ qualifier from pointer target type [enabled by default]
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Quiets clang warning:
xhost.c:812:21: warning: unused parameter 'sig' [-Wunused-parameter]
nameserver_lost(int sig)
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
wait
Win32 has neither SIGALRM nor sigaction(), so don't use SIGALRM to timeout
gethostaddr(), just wait
Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Reviewed-by: Colin Harrison <colin.harrison@virgin.net>
|
|
X_NOT_POSIX has never been defined by our autoconf scripts, only for a few
platforms in <X11/Xosdefs.h>, of which MinGW seems to be the most relevant
today, and since that doesn't have alarm() it doesn't need this code either.
First pass was done with 'unifdef -UX_NOT_POSIX', followed by
manual tweaking to adjust indent levels, etc.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jon TURNEY <jon.turney@dronecode.org.uk>
|
|
Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Reviewed-by: Colin Harrison <colin.harrison@virgin.net>
|
|
Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Reviewed-by: Colin Harrison <colin.harrison@virgin.net>
|
|
Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Reviewed-by: Colin Harrison <colin.harrison@virgin.net>
|
|
xhost.c:154:19: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
for (i = 0; i < FAMILIES; i++)
~ ^ ~~~~~~~~
xhost.c:310:15: warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long') to 'int' [-Wshorten-64-to-32]
namelen = strlen(name);
~ ^~~~~~~~~~~~
xhost.c:311:40: warning: implicit conversion changes signedness: 'int' to 'size_t' (aka 'unsigned long') [-Wsign-conversion]
if ((lname = (char *)malloc(namelen+1)) == NULL) {
~~~~~~ ~~~~~~~^~
xhost.c:707:46: warning: implicit conversion changes signedness: 'int' to 'socklen_t' (aka 'unsigned int') [-Wsign-conversion]
getnameinfo((struct sockaddr *) &saddr, saddrlen, inetname,
~~~~~~~~~~~ ^~~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Fixes gcc warnings:
xhost.c:290: warning: declaration of 'dpy' shadows a global declaration
xhost.c:160: warning: shadowed declaration is here
xhost.c:839: warning: declaration of 'dpy' shadows a global declaration
xhost.c:160: warning: shadowed declaration is here
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Drops use of autoconf's obsolete AC_TYPE_SIGNAL and the even more
obsolete Imake SIGNALRETURNSINT.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Has never been converted to build in modular builds, so has been unusable
since X11R7.0 release in 2005. All known platforms with TLI/XTI support
that X11R7 & later releases run on also have (and mostly prefer) BSD
socket support for their networking API.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Has never been converted to build in modular builds, so has been unusable
since X11R7.0 release in 2005. DNETCONN support was removed from xtrans
back in 2008.
Performed via "unifdef -UDNETCONN".
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Fixes gcc warning:
xhost.c:453:6: warning: declaration of ‘namelen’ shadows a previous local
xhost.c:339:9: warning: shadowed declaration is here
Also removes unnecessary malloc and memcpy by just using the string
pointers we already have, since XAddHost & XRemoveHost will copy the
specified length of text from the strings to the Xlib request buffer
before returning, not relying on nil termination.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
layout and comment the top portion of configure.ac
add missing AC_CONFIG_SRCDIR
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
However, server interpreted addresses can be used to achieve the same
behavior.
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Julien Cristau <jcristau@debian.org>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
|
|
xhost needs the Family definitions from <X11/Xauth.h> to build, but
doesn't call any libXau functions, so doesn't need to link against it.
Fixes Solaris ld -z guidance warning:
ld: guidance: removal of unused dependency recommended: libXau.so.6
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Reviewed-by: Gaetan Nadon <memsize@videotron.ca>
Reviewed-by: Jamey Sharp <jamey@minilop.net>
|
|
We don't need a display to show the usage message.
X.Org bug#39633 <https://bugs.freedesktop.org/show_bug.cgi?id=39633>
Signed-off-by: Julien Cristau <jcristau@debian.org>
|
|
In the ServerInterpreted case, XAddHost and XRemoveHost are capable of
failing when they lack request buffer memory. Notice this situation,
and report correctly.
Signed-off-by: Kees Cook <kees.cook@canonical.com>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Julien Cristau <jcristau@debian.org>
|
|
Use services provided by XORG_MANPAGE_SECTIONS.
Use standard Makefile for man pages.
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Using s/[ \t]*$//
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
This silences an Autoconf warning
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|