(xgetcwd): Reorganize to avoid some duplication.

Use an initial, malloc'd, buffer of length 128 rather than
a statically allocated one of length 1024.

diff --git a/lib/xgetcwd.c b/lib/xgetcwd.c
index 0af66fb88844942c178cf2b8295888a1af15005b..55e12afcd1d79d7d5854aa0e8e4f3ccadf13a38e 100644 (file)
--- a/lib/xgetcwd.c
+++ b/lib/xgetcwd.c
@@ -54,38 +54,25 @@ xgetcwd ()
 #if defined __GLIBC__ && __GLIBC__ >= 2
   return getcwd (NULL, 0);
 #else
-  char *ret;
-  size_t path_max;
-  char buf[1024];
+  size_t buf_size = 128;  /* must be a power of 2 */
+  char *buf = NULL;
 
-  errno = 0;
-  ret = getcwd (buf, sizeof (buf));
-  if (ret != NULL)
-    return xstrdup (buf);
-  if (errno != ERANGE)
-    return NULL;
-
-  path_max = 1 << 10;
-
-  for (;;)
+  while (1)
     {
-      char *cwd = (char *) xmalloc (path_max);
-      int save_errno;
+      char *ret;
+      buf = (char *) xrealloc (buf, buf_size);
 
-      errno = 0;
-      ret = getcwd (cwd, path_max);
-      if (ret != NULL)
-       return ret;
-      save_errno = errno;
-      free (cwd);
-      if (save_errno != ERANGE)
+      cwd = getcwd (buf, buf_size);
+      if (cwd != NULL)
+       return cwd;
+      if (errno != ERANGE)
        {
-         errno = save_errno;
+         free (buf);
          return NULL;
        }
 
-      path_max *= 2;
-      if (path_max == 0)
+      buf_size *= 2;
+      if (buf_size == 0)
        xalloc_die ();
     }
 #endif