debug log missing DISPLAY environment when X11 forwarding
authordjm <djm@openbsd.org>
Fri, 17 Apr 2015 13:16:48 +0000 (13:16 +0000)
committerdjm <djm@openbsd.org>
Fri, 17 Apr 2015 13:16:48 +0000 (13:16 +0000)
requested; bz#1682 ok dtucker@

usr.bin/ssh/ssh.c

index c4087c1..1b83d9d 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh.c,v 1.416 2015/03/03 06:48:58 djm Exp $ */
+/* $OpenBSD: ssh.c,v 1.417 2015/04/17 13:16:48 djm Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -1623,6 +1623,8 @@ ssh_session(void)
        }
        /* Request X11 forwarding if enabled and DISPLAY is set. */
        display = getenv("DISPLAY");
+       if (display == NULL && options.forward_x11)
+               debug("X11 forwarding requested but DISPLAY not set");
        if (options.forward_x11 && display != NULL) {
                char *proto, *data;
                /* Get reasonable local authentication information. */
@@ -1724,6 +1726,8 @@ ssh_session2_setup(int id, int success, void *arg)
                return; /* No need for error message, channels code sens one */
 
        display = getenv("DISPLAY");
+       if (display == NULL && options.forward_x11)
+               debug("X11 forwarding requested but DISPLAY not set");
        if (options.forward_x11 && display != NULL) {
                char *proto, *data;
                /* Get reasonable local authentication information. */