X-Git-Url: https://git.octo.it/?a=blobdiff_plain;ds=sidebyside;f=entry.c;h=15b8fda0c648d6dbb55eddde00e7accfeda5ac8e;hb=5bdac8b3269aea963c3192c93c26c1524def5464;hp=874516e174d0704808587689504b9ba84f2dc9af;hpb=b43d44779bf98977b211256f936d0edda8a9625a;p=git.git diff --git a/entry.c b/entry.c index 874516e1..15b8fda0 100644 --- a/entry.c +++ b/entry.c @@ -12,10 +12,10 @@ static void create_directories(const char *path, struct checkout *state) len = slash - path; memcpy(buf, path, len); buf[len] = 0; - if (mkdir(buf, 0755)) { + if (mkdir(buf, 0777)) { if (errno == EEXIST) { struct stat st; - if (len > state->base_dir_len && state->force && !unlink(buf) && !mkdir(buf, 0755)) + if (len > state->base_dir_len && state->force && !unlink(buf) && !mkdir(buf, 0777)) continue; if (!stat(buf, &st) && S_ISDIR(st.st_mode)) continue; /* ok */ @@ -57,19 +57,10 @@ static void remove_subtree(const char *path) die("cannot rmdir %s", path); } -static int create_file(const char *path, unsigned int mode, int force) +static int create_file(const char *path, unsigned int mode) { - int fd; - mode = (mode & 0100) ? 0777 : 0666; - fd = open(path, O_WRONLY | O_TRUNC | O_CREAT, mode); - if (fd < 0) { - if (errno == EISDIR && force) { - remove_subtree(path); - fd = open(path, O_WRONLY | O_TRUNC | O_CREAT, mode); - } - } - return fd; + return open(path, O_WRONLY | O_TRUNC | O_CREAT | O_EXCL, mode); } static int write_entry(struct cache_entry *ce, const char *path, struct checkout *state) @@ -90,7 +81,7 @@ static int write_entry(struct cache_entry *ce, const char *path, struct checkout } switch (ntohl(ce->ce_mode) & S_IFMT) { case S_IFREG: - fd = create_file(path, ntohl(ce->ce_mode), state->force); + fd = create_file(path, ntohl(ce->ce_mode)); if (fd < 0) { free(new); return error("git-checkout-cache: unable to create file %s (%s)", @@ -151,6 +142,11 @@ int checkout_entry(struct cache_entry *ce, struct checkout *state) * just do the right thing) */ unlink(path); + if (S_ISDIR(st.st_mode)) { + if (!state->force) + return error("%s is a directory", path); + remove_subtree(path); + } } else if (state->not_new) return 0; create_directories(path, state);