12533: pathconf() without tail in mkdir

2 files changed, 21 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index 73c9e0543..39e71f5d4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2000-08-04  Clint Adams  <schizo@debian.org>
+
+	* 12533: Src/Modules/files.c: drop the tail of the path
+	passed to mkdir in the call to pathconf().
+
 2000-08-04  Sven Wischnowsky  <wischnow@zsh.org>
 
 	* 12525: Src/module.c: remove (wrong) test for linked-in modules
diff --git a/Src/Modules/files.c b/Src/Modules/files.c
index d01a59b03..ac775c75a 100644
--- a/Src/Modules/files.c
+++ b/Src/Modules/files.c
@@ -71,6 +71,7 @@ bin_mkdir(char *nam, char **args, char *ops, int func)
     mode_t oumask = umask(0);
     mode_t mode = 0777 & ~oumask;
     int err = 0;
+    char *head;
 
     umask(oumask);
     if(ops['m']) {
@@ -91,8 +92,19 @@ bin_mkdir(char *nam, char **args, char *ops, int func)
 
 	while(ptr > *args + (**args == '/') && *--ptr == '/')
 	    *ptr = 0;
-	if(zpathmax(unmeta(*args)) < 0) {
-	    zwarnnam(nam, "%s: %e", *args, errno);
+
+/* Drop the tail so that pathconf receives a potentially valid pathname */
+	head = (char *) ztrdup(*args);
+	if ((ptr = strrchr(head, '/')))
+	    *ptr = 0;
+	else {
+/* Relative to current directory */
+	    *head = '.';
+	    *(head + 1) = '\0';
+	}
+
+	if(zpathmax(unmeta(head)) < 0) {
+	    zwarnnam(nam, "%s: %e", head, errno);
 	    err = 1;
 	    continue;
 	}
@@ -121,6 +133,8 @@ bin_mkdir(char *nam, char **args, char *ops, int func)
 	    }
 	} else
 	    err |= domkdir(nam, *args, mode, 0);
+
+	free(head);
     }
     return err;
 }