X-Git-Url: http://git.marmaro.de/?a=blobdiff_plain;f=sbr%2Fmakedir.c;h=a76f9108304a0a99ceb9119778bf57b20ed7669b;hb=2968a0027b4c64664076f0d03a37f9744fe26782;hp=6cf9e03c29303f90aab20eb9612228cbd78c5760;hpb=1691e80890e5d8ba258c51c214a3e91880e1db2b;p=mmh

diff --git a/sbr/makedir.c b/sbr/makedir.c
index 6cf9e03..a76f910 100644
--- a/sbr/makedir.c
+++ b/sbr/makedir.c
@@ -1,79 +1,75 @@
-
 /*
- * makedir.c -- make a directory
- *
- * $Id$
- */
+** makedir.c -- make a directory
+**
+** This code is Copyright (c) 2002, by the authors of nmh.  See the
+** COPYRIGHT file in the root directory of the nmh distribution for
+** complete copyright information.
+*/
 
 /*
- * Modified to try recursive create.
- */
+** Modified to try recursive create.
+*/
 
 #include <h/mh.h>
 #include <errno.h>
 #include <sys/param.h>
 #include <sys/file.h>
 
-extern int errno;
-	
 int
-makedir (char *dir)
+makedir(char *dir)
 {
-    pid_t pid;
-    register char *cp;
-    register char *c;
-    char path[PATH_MAX];
+	char path[PATH_MAX];
+	char* folder_perms_ASCII;
+	int had_an_error = 0;
+	mode_t folder_perms, saved_umask;
+	register char* c;
+
+	context_save();  /* save the context file */
+	fflush(stdout);
 
-    context_save();	/* save the context file */
-    fflush(stdout);
+	if (!(folder_perms_ASCII = context_find("folder-protect")))
+		folder_perms_ASCII = foldprot;  /* defaults to "700" */
 
-    if (getuid () == geteuid ()) {
-	    c = strncpy(path, dir, sizeof(path));     
+	/*
+	** Because mh-profile.man documents "Folder-Protect:" as an octal
+	** constant, and we don't want to force the user to remember to
+	** include a leading zero, we call atooi(folder_perms_ASCII) here
+	** rather than strtoul(folder_perms_ASCII, NULL, 0).  Therefore,
+	** if anyone ever tries to specify a mode in say, hex, they'll
+	** get garbage.  (I guess nmh uses its atooi() function rather
+	** than calling strtoul() with a radix of 8 because some ancient
+	** platforms are missing that functionality.
+	*/
+	folder_perms = atooi(folder_perms_ASCII);
 
-	    while ((c = strchr((c + 1), '/')) != NULL) {	
-		    *c = (char)0;
-		    if (access(path, X_OK)) {
-			    if (errno != ENOENT){
-				    advise (dir, "unable to create directory");
-				    return 0;
-			    }			    
-			    if (mkdir(path, 0775)) {
-				    advise (dir, "unable to create directory");
-				    return 0;
-			    }
-		    }
-		    *c = '/';
-	    }
- 
-	    if (mkdir (dir, 0755) == -1) {
-		    advise (dir, "unable to create directory");
-		    return 0;
-	    }
-    } else {
-	switch (pid = vfork()) {
-	case -1: 
-	    advise ("fork", "unable to");
-	    return 0;
+	/*
+	** Folders have definite desired permissions that are set -- we
+	** don't want to interact with the umask.  Clear it temporarily.
+	*/
+	saved_umask = umask(0);
 
-	case 0: 
-	    setgid (getgid ());
-	    setuid (getuid ());
+	c = strncpy(path, dir, sizeof(path));
 
-	    execl ("/bin/mkdir", "mkdir", dir, NULL);
-	    execl ("/usr/bin/mkdir", "mkdir", dir, NULL);
-	    fprintf (stderr, "unable to exec ");
-	    perror ("mkdir");
-	    _exit (-1);
+	while (!had_an_error && (c = strchr((c + 1), '/')) != NULL) {
+		*c = '\0';
+		/* Create an outer directory. */
+		if (mkdir(path, folder_perms) == -1 &&
+				errno != EEXIST) {
+			advise(dir, "unable to create directory");
+			had_an_error = 1;
+		}
+		*c = '/';
+	}
 
-	default: 
-	    if (pidXwait(pid, "mkdir"))
-		return 0;
-	    break;
+	/*
+	** Create the innermost nested subdirectory of the
+	** path we're being asked to create.
+	*/
+	if (!had_an_error && mkdir(dir, folder_perms)==-1) {
+		advise(dir, "unable to create directory");
+		had_an_error = 1;
 	}
-    }
+	umask(saved_umask);  /* put the user's umask back */
 
-    if (!(cp = context_find ("folder-protect")))
-	cp = foldprot;
-    chmod (dir, atooi (cp));
-    return 1;
+	return (had_an_error) ? 0 : 1;
 }