]> git.ipfire.org Git - thirdparty/lxc.git/commitdiff
Revert "utils: reimplement/fix mkdir_p()"
authorStéphane Graber <stgraber@ubuntu.com>
Thu, 18 Apr 2013 08:29:44 +0000 (10:29 +0200)
committerStéphane Graber <stgraber@ubuntu.com>
Thu, 18 Apr 2013 08:29:44 +0000 (10:29 +0200)
This reverts commit 8de4140644f01180f2fdab55b0ab0f13d1c761c6.

This commit was preventing container startup on my machine, making them
all fail with various "No such file or directory" errors.

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
src/lxc/utils.c

index 979455347a5dc81594dd54959344d1bc3deacf41..e07ca7b13a88a2f84b3c2813d9318e55b34aa4b6 100644 (file)
@@ -95,25 +95,39 @@ extern int get_u16(unsigned short *val, const char *arg, int base)
        return 0;
 }
 
+static int is_all_slashes(char *path)
+{
+       while (*path && *path == '/')
+               path++;
+       if (*path)
+               return 0;
+       return 1;
+}
+
 extern int mkdir_p(char *dir, mode_t mode)
 {
-       char *tmp = dir;
-       char *orig = dir;
-       char *makeme;
-
-       do {
-               dir = tmp + strspn(tmp, "/");
-               tmp = dir + strcspn(dir, "/");
-               makeme = strndupa(orig, dir - orig);
-               if (*makeme) {
-                       if (!access(makeme, F_OK))
-                               return 0;
-                       if (mkdir(makeme, mode)) {
-                               SYSERROR("failed to create directory '%s'\n", makeme);
-                               return -1;
-                       }
-               }
-       } while(tmp != dir);
+       int ret;
+       char *d;
+
+       if (is_all_slashes(dir))
+               return 0;
+
+       d = strdup(dir);
+       if (!d)
+               return -1;
+
+       ret = mkdir_p(dirname(d), mode);
+       free(d);
+       if (ret)
+               return -1;
+
+       if (!access(dir, F_OK))
+               return 0;
+
+       if (mkdir(dir, mode)) {
+               SYSERROR("failed to create directory '%s'\n", dir);
+               return -1;
+       }
 
        return 0;
 }