Junio C Hamano <gitster@xxxxxxxxx> writes: >> @@ -2281,6 +2287,7 @@ static const char *unpack(int err_fd, struct shallow_info *si) >> if (status) >> return "index-pack fork failed"; >> pack_lockfile = index_pack_lockfile(child.out, NULL); >> + atexit(unlink_pack_lockfile); > > Hmph, why isn't this a straight application of tempfile API? Ah, sorry, that has already been raised in the thread. I should have checked first. Sorry for the noise.