diff options
author | Lars Schneider <larsxschneider@gmail.com> | 2017-10-05 13:44:06 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-10-05 14:01:07 +0300 |
commit | 03b95333db32ea92b8a4a61f2f055a900f8d7c84 (patch) | |
tree | a64e7498ca0fd061b3ade90c1fac35cff60ae9c8 | |
parent | 2841e8f81cb2820024804b9341577be1d0ce1240 (diff) |
entry.c: update cache entry only for existing files
In 2841e8f ("convert: add "status=delayed" to filter process protocol",
2017-06-30) we taught the filter process protocol to delay responses.
That means an external filter might answer in the first write_entry()
call on a file that requires filtering "I got your request, but I
can't answer right now. Ask again later!". As Git got no answer, we do
not write anything to the filesystem. Consequently, the lstat() call in
the finish block of the function writes garbage to the cache entry.
The garbage is eventually overwritten when the filter answers with
the final file content in a subsequent write_entry() call.
Fix the brief time window of garbage in the cache entry by adding a
special finish block that does nothing for delayed responses. The cache
entry is written properly in a subsequent write_entry() call where
the filter responds with the final file content.
Reported-by: Jeff King <peff@peff.net>
Signed-off-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | entry.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -290,7 +290,7 @@ static int write_entry(struct cache_entry *ce, ce->name, new, size, &buf, dco); if (ret && string_list_has_string(&dco->paths, ce->name)) { free(new); - goto finish; + goto delayed; } } else ret = convert_to_working_tree( @@ -346,6 +346,7 @@ finish: ce->ce_flags |= CE_UPDATE_IN_BASE; state->istate->cache_changed |= CE_ENTRY_CHANGED; } +delayed: return 0; } |