A zero-length return from platform_get_x_display() (for instance, a
authorjacob <jacob@cda61777-01e9-0310-a592-d414129be87e>
Sat, 11 Feb 2006 18:00:23 +0000 (18:00 +0000)
committerjacob <jacob@cda61777-01e9-0310-a592-d414129be87e>
Sat, 11 Feb 2006 18:00:23 +0000 (18:00 +0000)
zero-length DISPLAY variable in the environment) caused an assertion
failure when X11 forwarding was attempted. Fixed (now treated the same
as a NULL return, e.g., a non-existent DISPLAY variable).

git-svn-id: svn://svn.tartarus.org/sgt/putty@6549 cda61777-01e9-0310-a592-d414129be87e

x11fwd.c

index 13c6823..6e04abe 100644 (file)
--- a/x11fwd.c
+++ b/x11fwd.c
@@ -245,7 +245,7 @@ char *x11_display(const char *display) {
     char *ret;
     if(!display || !*display) {
        /* try to find platform-specific local display */
-       if((ret = platform_get_x_display())==0)
+       if((ret = platform_get_x_display())==0 || !*ret)
            /* plausible default for all platforms */
            ret = dupstr(":0");
     } else