summary refs log tree commit diff
diff options
context:
space:
mode:
authorClint Adams <clint@users.sourceforge.net>2000-09-22 22:16:15 +0000
committerClint Adams <clint@users.sourceforge.net>2000-09-22 22:16:15 +0000
commit2a625db39df5e5e5f72556f0cd4a02005115c50f (patch)
tree9eb6be89333525ea9c74c14887da7a37aa8739cb
parent430e2947951207e45b02ba1ebc04586d06ade021 (diff)
downloadzsh-2a625db39df5e5e5f72556f0cd4a02005115c50f.tar.gz
zsh-2a625db39df5e5e5f72556f0cd4a02005115c50f.tar.xz
zsh-2a625db39df5e5e5f72556f0cd4a02005115c50f.zip
12859: dynamically-allocate buffer in ztat(), ztrdupstring()
-rw-r--r--ChangeLog6
-rw-r--r--Src/Zle/compresult.c14
-rw-r--r--Src/string.c20
3 files changed, 32 insertions, 8 deletions
diff --git a/ChangeLog b/ChangeLog
index acb35ef81..7281799b8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2000-09-22  Clint Adams  <schizo@debian.org>
+
+	* 12859: Src/string.c, Src/Zle/compresult.c:
+	dynamically allocate buffer in ztat, remove
+	duplication loop to ztrdupstrip().
+
 2000-09-20  Bart Schaefer  <schaefer@zsh.org>
 
 	* 12851, 12852: Completion/Commands/_expand_word,
diff --git a/Src/Zle/compresult.c b/Src/Zle/compresult.c
index 19d785bb8..92bb2c803 100644
--- a/Src/Zle/compresult.c
+++ b/Src/Zle/compresult.c
@@ -731,19 +731,17 @@ do_ambiguous(void)
 mod_export int
 ztat(char *nam, struct stat *buf, int ls)
 {
-    char b[PATH_MAX], *p;
+    int e;
+    char *b;
 
     if (!(ls ? lstat(nam, buf) : stat(nam, buf)))
 	return 0;
 
-    for (p = b; p < b + sizeof(b) - 1 && *nam; nam++)
-	if (*nam == '\\' && nam[1])
-	    *p++ = *++nam;
-	else
-	    *p++ = *nam;
-    *p = '\0';
+    b = ztrdupstrip(nam, '\\');
 
-    return ls ? lstat(b, buf) : stat(b, buf);
+    e = ls ? lstat(b, buf) : stat(b, buf);
+    zsfree(b);
+    return e;
 }
 
 /* Insert a single match in the command line. */
diff --git a/Src/string.c b/Src/string.c
index 3dad89911..57775359e 100644
--- a/Src/string.c
+++ b/Src/string.c
@@ -133,3 +133,23 @@ appstr(char *base, char const *append)
 {
     return strcat(realloc(base, strlen(base) + strlen(append) + 1), append);
 }
+
+/* Duplicate a string, stripping delimiters. */
+
+/**/
+mod_export char *
+ztrdupstrip(const char *nam, char delim)
+{
+    char *p, *buf;
+
+    buf = zalloc(strlen(nam));
+
+    for (p = buf; *nam; nam++)
+        if (*nam == delim && nam[1])
+            *p++ = *++nam;
+        else
+            *p++ = *nam;
+    *p = '\0';
+
+    return buf;
+}