Note Earl's regression fix for SSL_CTX_clear_options() on older OpenSSL.

[~andy/fetchmail] / env.c

diff --git a/env.c b/env.c
index 9b9e608cfafbf5410754d1c3df4a1918687874d4..edf998983a88be540d639946ed9c8b669711f07f 100644 (file)
--- a/env.c
+++ b/env.c
@@ -30,7 +30,9 @@
 #include <locale.h>
 #endif
 
+#ifndef HAVE_DECL_GETENV
 extern char *getenv(const char *);     /* needed on sysV68 R3V7.1. */
+#endif
 
 void envquery(int argc, char **argv)
 /* set up basic stuff from the environment (including the rc file name) */
@@ -105,10 +107,12 @@ void envquery(int argc, char **argv)
        user = xstrdup(pwp->pw_name);
     }
 
+    endpwent();
+
     /* compute user's home directory */
     home = getenv("HOME_ETC");
     if (!home && !(home = getenv("HOME")))
-       home = pwp->pw_dir;
+       home = xstrdup(pwp->pw_dir);
 
     /* compute fetchmail's home directory */
     if (!(fmhome = getenv("FETCHMAILHOME")))
@@ -159,7 +163,7 @@ char *host_fqdn(int required)
        memset(&hints, 0, sizeof hints);
        hints.ai_family = AF_UNSPEC;
        hints.ai_socktype = SOCK_STREAM;
-       hints.ai_flags=AI_CANONNAME;
+       hints.ai_flags = AI_CANONNAME;
 
        e = fm_getaddrinfo(tmpbuf, NULL, &hints, &res);
        if (e) {
@@ -176,7 +180,7 @@ char *host_fqdn(int required)
            }
        }
 
-       result = xstrdup(res->ai_canonname);
+       result = xstrdup(res->ai_canonname ? res->ai_canonname : tmpbuf);
        fm_freeaddrinfo(res);
     }
     else
@@ -290,9 +294,9 @@ char *visbuf(const char *buf)
 
     needed = strlen(buf) * 5 + 1; /* worst case: HEX, plus NUL byte */
 
-    if (needed > vbufs) {
+    if (!vbuf || needed > vbufs) {
        vbufs = needed;
-       vbuf = xrealloc(vbuf, vbufs);
+       vbuf = (char *)xrealloc(vbuf, vbufs);
     }
 
     tp = vbuf;
@@ -318,7 +322,7 @@ char *visbuf(const char *buf)
            buf++;
        }
     }
-    *tp++ = '\0';
+    *tp = '\0';
     return(vbuf);
 }
 /* env.c ends here */