aboutsummaryrefslogtreecommitdiffstats
path: root/cache.c
diff options
context:
space:
mode:
authorJohn Keeping <john@keeping.me.uk>2017-04-24 19:38:34 +0100
committerJohn Keeping <john@keeping.me.uk>2017-10-03 19:19:34 +0100
commit3b485cc5422f800d142c7023295e82c0a1c10b19 (patch)
treec5c46cdf02c2cff650cc15342c3a5ced861c912c /cache.c
parentde297883385030f4bdcd2f5c3839d1187611b949 (diff)
downloadcgit-3b485cc5422f800d142c7023295e82c0a1c10b19.tar.gz
cgit-3b485cc5422f800d142c7023295e82c0a1c10b19.tar.bz2
cgit-3b485cc5422f800d142c7023295e82c0a1c10b19.zip
cache: flush stdio before restoring FDs
As described in commit 2efb59e (ui-patch: Flush stdout after outputting data, 2014-06-11), we need to ensure that stdout is flushed before restoring the file descriptor when writing to the cache. It turns out that it's not just ui-patch that is affected by this but also raw diff which writes to stdout internally. Let's avoid risking more places doing this by ensuring that stdout is flushed after writing in fill_slot(). Signed-off-by: John Keeping <john@keeping.me.uk>
Diffstat (limited to 'cache.c')
-rw-r--r--cache.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/cache.c b/cache.c
index 6736a01..2ccdc4e 100644
--- a/cache.c
+++ b/cache.c
@@ -224,6 +224,12 @@ static int fill_slot(struct cache_slot *slot)
/* Generate cache content */
slot->fn();
+ /* Make sure any buffered data is flushed to the file */
+ if (fflush(stdout)) {
+ close(tmp);
+ return errno;
+ }
+
/* update stat info */
if (fstat(slot->lock_fd, &slot->cache_st)) {
close(tmp);