bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: tmpdir.c:path_search() v. VMS


From: Paul Eggert
Subject: Re: tmpdir.c:path_search() v. VMS
Date: Mon, 15 Jul 2013 09:46:18 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7

Thanks for the heads up.  I pushed this but have not tested
under VMS, so please give it a try.

---
 ChangeLog    | 12 ++++++++++++
 lib/tmpdir.c | 22 ++++++++++++++++------
 2 files changed, 28 insertions(+), 6 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 4d73a26..6929de0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2013-07-15  Paul Eggert  <address@hidden>
+
+       tmpdir: port to VMS, to // != /, and to long dirs
+       * lib/tmpdir.c (__libc_secure_getenv) [!_LIBC]: Rename from
+       __secure_getenv, so that we're more like the glibc version.
+       All uses changed.
+       (path_search): Don't put slash after directory if __VMS.
+       Problem reported by Steven M. Schweda in
+       <http://lists.gnu.org/archive/html/bug-gnulib/2013-07/msg00019.html>.
+       Do not remove trailing slash from "//".
+       Do not assume dlen <= INT_MAX.
+
 2013-07-09  Paul Eggert  <address@hidden>
 
        regex: port to --with-included-regex --enable-gcc-warnings non-threaded
diff --git a/lib/tmpdir.c b/lib/tmpdir.c
index e804416..aa2164d 100644
--- a/lib/tmpdir.c
+++ b/lib/tmpdir.c
@@ -53,7 +53,7 @@
 # define struct_stat64 struct stat64
 #else
 # define struct_stat64 struct stat
-# define __secure_getenv secure_getenv
+# define __libc_secure_getenv secure_getenv
 # define __xstat64(version, path, buf) stat (path, buf)
 #endif
 
@@ -89,6 +89,11 @@ path_search (char *tmpl, size_t tmpl_len, const char *dir, 
const char *pfx,
 {
   const char *d;
   size_t dlen, plen;
+#ifdef __VMS
+  bool add_slash = false;
+#else
+  bool add_slash = true;
+#endif
 
   if (!pfx || !pfx[0])
     {
@@ -104,7 +109,7 @@ path_search (char *tmpl, size_t tmpl_len, const char *dir, 
const char *pfx,
 
   if (try_tmpdir)
     {
-      d = __secure_getenv ("TMPDIR");
+      d = __libc_secure_getenv ("TMPDIR");
       if (d != NULL && direxists (d))
         dir = d;
       else if (dir != NULL && direxists (dir))
@@ -138,17 +143,22 @@ path_search (char *tmpl, size_t tmpl_len, const char 
*dir, const char *pfx,
         }
     }
 
+  /* Remove trailing slashes, except remove just one from "//".  */
   dlen = strlen (dir);
-  while (dlen >= 1 && ISSLASH (dir[dlen - 1]))
-    dlen--;                     /* remove trailing slashes */
+  if (dlen == 2 && ISSLASH (dir[0]) && ISSLASH (dir[1]))
+    dlen--;
+  else
+    while (0 < dlen && ISSLASH (dir[dlen - 1]))
+      dlen--;
 
   /* check we have room for "${dir}/${pfx}XXXXXX\0" */
-  if (tmpl_len < dlen + 1 + plen + 6 + 1)
+  if (tmpl_len < dlen + add_slash + plen + 6 + 1)
     {
       __set_errno (EINVAL);
       return -1;
     }
 
-  sprintf (tmpl, "%.*s/%.*sXXXXXX", (int) dlen, dir, (int) plen, pfx);
+  memcpy (tmpl, dir, dlen);
+  sprintf (tmpl + dlen, &"/%.*sXXXXXX"[!add_slash], pfx, (int) plen);
   return 0;
 }
-- 
1.7.11.7




reply via email to

[Prev in Thread] Current Thread [Next in Thread]