René Scharfe <l.s.r@xxxxxx> writes: > Am 10.06.22 um 16:46 schrieb Han Xin: >> From: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> >> >> Change write_loose_object() to do an fsync() and close() before the >> oideq() sanity check at the end. This change re-joins code that was >> split up by the die() sanity check added in 748af44c63e (sha1_file: be >> paranoid when creating loose objects, 2010-02-21). >> >> I don't think that this change matters in itself, if we called die() >> it was possible that our data wouldn't fully make it to disk, but in >> any case we were writing data that we'd consider corrupted. It's >> possible that a subsequent "git fsck" will be less confused now. > > This is done before renaming the file, so git fsck is going to see (at > most) a tmp_obj_?????? file, which it ignores in either case, right? Yes, I thought I pointed that out in my review on the previous round, but I missed that it was still here in this round X-<. Thanks for noticing.
