summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormillert@openbsd.org <millert@openbsd.org>2016-02-01 21:18:17 +0000
committerDamien Miller <djm@mindrot.org>2016-02-05 10:44:42 +1100
commit5658ef2501e785fbbdf5de2dc33b1ff7a4dca73a (patch)
treebccb7513f721d70d8ac7512781cae0f39b58fd8f
parent537f88ec7bcf40bd444ac5584c707c5588c55c43 (diff)
upstream commit
Avoid ugly "DISPLAY "(null)" invalid; disabling X11 forwarding" message when DISPLAY is not set. This could also result in a crash on systems with a printf that doesn't handle NULL. OK djm@ Upstream-ID: 20ee0cfbda678a247264c20ed75362042b90b412
-rw-r--r--clientloop.c7
1 files changed, 4 insertions, 3 deletions
diff --git a/clientloop.c b/clientloop.c
index f8f9a3f2..f0a08f23 100644
--- a/clientloop.c
+++ b/clientloop.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: clientloop.c,v 1.282 2016/01/29 23:04:46 dtucker Exp $ */
+/* $OpenBSD: clientloop.c,v 1.283 2016/02/01 21:18:17 millert Exp $ */
/*
* Author: Tatu Ylonen <ylo@cs.hut.fi>
* Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -318,8 +318,9 @@ client_x11_get_proto(const char *display, const char *xauth_path,
proto[0] = data[0] = xauthfile[0] = xauthdir[0] = '\0';
if (!client_x11_display_valid(display)) {
- logit("DISPLAY \"%s\" invalid; disabling X11 forwarding",
- display);
+ if (display != NULL)
+ logit("DISPLAY \"%s\" invalid; disabling X11 forwarding",
+ display);
return -1;
}
if (xauth_path != NULL && stat(xauth_path, &st) == -1) {