Merge branch 'jc/cache-tree' into next
authorJunio C Hamano <junkio@cox.net>
Sun, 28 May 2006 20:39:51 +0000 (13:39 -0700)
committerJunio C Hamano <junkio@cox.net>
Sun, 28 May 2006 20:39:51 +0000 (13:39 -0700)
* jc/cache-tree:
git-write-tree writes garbage on sparc64

1  2 
read-cache.c
diff --combined read-cache.c
index 36bd4ea0be84516be217945840a78c49872bfa38,5270e834dd06974f77f0dcdfbee3eaea64249129..c499c5185678b6b4ac35d57d1ab0b36cc2ff056b
@@@ -346,70 -346,6 +346,70 @@@ int ce_path_match(const struct cache_en
        return 0;
  }
  
 +/*
 + * We fundamentally don't like some paths: we don't want
 + * dot or dot-dot anywhere, and for obvious reasons don't
 + * want to recurse into ".git" either.
 + *
 + * Also, we don't want double slashes or slashes at the
 + * end that can make pathnames ambiguous.
 + */
 +static int verify_dotfile(const char *rest)
 +{
 +      /*
 +       * The first character was '.', but that
 +       * has already been discarded, we now test
 +       * the rest.
 +       */
 +      switch (*rest) {
 +      /* "." is not allowed */
 +      case '\0': case '/':
 +              return 0;
 +
 +      /*
 +       * ".git" followed by  NUL or slash is bad. This
 +       * shares the path end test with the ".." case.
 +       */
 +      case 'g':
 +              if (rest[1] != 'i')
 +                      break;
 +              if (rest[2] != 't')
 +                      break;
 +              rest += 2;
 +      /* fallthrough */
 +      case '.':
 +              if (rest[1] == '\0' || rest[1] == '/')
 +                      return 0;
 +      }
 +      return 1;
 +}
 +
 +int verify_path(const char *path)
 +{
 +      char c;
 +
 +      goto inside;
 +      for (;;) {
 +              if (!c)
 +                      return 1;
 +              if (c == '/') {
 +inside:
 +                      c = *path++;
 +                      switch (c) {
 +                      default:
 +                              continue;
 +                      case '/': case '\0':
 +                              break;
 +                      case '.':
 +                              if (verify_dotfile(path))
 +                                      continue;
 +                      }
 +                      return 0;
 +              }
 +              c = *path++;
 +      }
 +}
 +
  /*
   * Do we have another file that has the beginning components being a
   * proper superset of the name we're trying to add?
@@@ -551,8 -487,6 +551,8 @@@ int add_cache_entry(struct cache_entry 
  
        if (!ok_to_add)
                return -1;
 +      if (!verify_path(ce->name))
 +              return -1;
  
        if (!skip_df_check &&
            check_file_directory_conflict(ce, pos, ok_to_replace)) {
        return 0;
  }
  
 +/* Three functions to allow overloaded pointer return; see linux/err.h */
 +static inline void *ERR_PTR(long error)
 +{
 +      return (void *) error;
 +}
 +
 +static inline long PTR_ERR(const void *ptr)
 +{
 +      return (long) ptr;
 +}
 +
 +static inline long IS_ERR(const void *ptr)
 +{
 +      return (unsigned long)ptr > (unsigned long)-1000L;
 +}
 +
 +/*
 + * "refresh" does not calculate a new sha1 file or bring the
 + * cache up-to-date for mode/content changes. But what it
 + * _does_ do is to "re-match" the stat information of a file
 + * with the cache, so that you can refresh the cache for a
 + * file that hasn't been changed but where the stat entry is
 + * out of date.
 + *
 + * For example, you'd want to do this after doing a "git-read-tree",
 + * to link up the stat cache details with the proper files.
 + */
 +static struct cache_entry *refresh_entry(struct cache_entry *ce, int really)
 +{
 +      struct stat st;
 +      struct cache_entry *updated;
 +      int changed, size;
 +
 +      if (lstat(ce->name, &st) < 0)
 +              return ERR_PTR(-errno);
 +
 +      changed = ce_match_stat(ce, &st, really);
 +      if (!changed) {
 +              if (really && assume_unchanged &&
 +                  !(ce->ce_flags & htons(CE_VALID)))
 +                      ; /* mark this one VALID again */
 +              else
 +                      return NULL;
 +      }
 +
 +      if (ce_modified(ce, &st, really))
 +              return ERR_PTR(-EINVAL);
 +
 +      size = ce_size(ce);
 +      updated = xmalloc(size);
 +      memcpy(updated, ce, size);
 +      fill_stat_cache_info(updated, &st);
 +
 +      /* In this case, if really is not set, we should leave
 +       * CE_VALID bit alone.  Otherwise, paths marked with
 +       * --no-assume-unchanged (i.e. things to be edited) will
 +       * reacquire CE_VALID bit automatically, which is not
 +       * really what we want.
 +       */
 +      if (!really && assume_unchanged && !(ce->ce_flags & htons(CE_VALID)))
 +              updated->ce_flags &= ~htons(CE_VALID);
 +
 +      return updated;
 +}
 +
 +int refresh_cache(unsigned int flags)
 +{
 +      int i;
 +      int has_errors = 0;
 +      int really = (flags & REFRESH_REALLY) != 0;
 +      int allow_unmerged = (flags & REFRESH_UNMERGED) != 0;
 +      int quiet = (flags & REFRESH_QUIET) != 0;
 +      int not_new = (flags & REFRESH_IGNORE_MISSING) != 0;
 +
 +      for (i = 0; i < active_nr; i++) {
 +              struct cache_entry *ce, *new;
 +              ce = active_cache[i];
 +              if (ce_stage(ce)) {
 +                      while ((i < active_nr) &&
 +                             ! strcmp(active_cache[i]->name, ce->name))
 +                              i++;
 +                      i--;
 +                      if (allow_unmerged)
 +                              continue;
 +                      printf("%s: needs merge\n", ce->name);
 +                      has_errors = 1;
 +                      continue;
 +              }
 +
 +              new = refresh_entry(ce, really);
 +              if (!new)
 +                      continue;
 +              if (IS_ERR(new)) {
 +                      if (not_new && PTR_ERR(new) == -ENOENT)
 +                              continue;
 +                      if (really && PTR_ERR(new) == -EINVAL) {
 +                              /* If we are doing --really-refresh that
 +                               * means the index is not valid anymore.
 +                               */
 +                              ce->ce_flags &= ~htons(CE_VALID);
 +                              active_cache_changed = 1;
 +                      }
 +                      if (quiet)
 +                              continue;
 +                      printf("%s: needs update\n", ce->name);
 +                      has_errors = 1;
 +                      continue;
 +              }
 +              active_cache_changed = 1;
 +              /* You can NOT just free active_cache[i] here, since it
 +               * might not be necessarily malloc()ed but can also come
 +               * from mmap(). */
 +              active_cache[i] = new;
 +      }
 +      return has_errors;
 +}
 +
  static int verify_hdr(struct cache_header *hdr, unsigned long size)
  {
        SHA_CTX c;
@@@ -766,7 -583,7 +766,7 @@@ int read_cache(void
  
        active_nr = ntohl(hdr->hdr_entries);
        active_alloc = alloc_nr(active_nr);
 -      active_cache = calloc(active_alloc, sizeof(struct cache_entry *));
 +      active_cache = xcalloc(active_alloc, sizeof(struct cache_entry *));
  
        offset = sizeof(*hdr);
        for (i = 0; i < active_nr; i++) {
@@@ -826,7 -643,7 +826,7 @@@ static int ce_write(SHA_CTX *context, i
  }
  
  static int write_index_ext_header(SHA_CTX *context, int fd,
-                                 unsigned long ext, unsigned long sz)
+                                 unsigned int ext, unsigned int sz)
  {
        ext = htonl(ext);
        sz = htonl(sz);