diff options
author | Jaime Soriano Pastor <jsorianopastor@gmail.com> | 2014-08-29 10:54:41 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-08-29 10:05:14 -0700 |
commit | 15999d0be8179fb7a2e6eafb931d25ed65df50aa (patch) | |
tree | 0b83f9709af847397e90d505c774b97397d523d4 | |
parent | 32f56600bb6ac6fc57183e79d2c1515dfa56672f (diff) | |
download | git-15999d0be8179fb7a2e6eafb931d25ed65df50aa.tar.gz |
read_index_from(): catch out of order entries when reading an index file
Signed-off-by: Jaime Soriano Pastor <jsorianopastor@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | read-cache.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/read-cache.c b/read-cache.c index 7f5645e745..22b0add52f 100644 --- a/read-cache.c +++ b/read-cache.c @@ -1438,6 +1438,21 @@ static struct cache_entry *create_from_disk(struct ondisk_cache_entry *ondisk, return ce; } +static void check_ce_order(struct cache_entry *ce, struct cache_entry *next_ce) +{ + int name_compare = strcmp(ce->name, next_ce->name); + if (0 < name_compare) + die("unordered stage entries in index"); + if (!name_compare) { + if (!ce_stage(ce)) + die("multiple stage entries for merged file '%s'", + ce->name); + if (ce_stage(ce) > ce_stage(next_ce)) + die("unordered stage entries for '%s'", + ce->name); + } +} + /* remember to discard_cache() before reading a different cache! */ int read_index_from(struct index_state *istate, const char *path) { @@ -1499,6 +1514,9 @@ int read_index_from(struct index_state *istate, const char *path) ce = create_from_disk(disk_ce, &consumed, previous_name); set_index_entry(istate, i, ce); + if (i > 0) + check_ce_order(istate->cache[i - 1], ce); + src_offset += consumed; } strbuf_release(&previous_name_buf); |