summaryrefslogtreecommitdiff
path: root/builtin-verify-pack.c
diff options
context:
space:
mode:
authorNicolas Pitre <nico@cam.org>2008-02-28 00:25:18 -0500
committerJunio C Hamano <gitster@pobox.com>2008-03-01 01:44:45 -0800
commit340814636dde3cbd2e461b12f9ae832d2100766a (patch)
treedfe72ebd2496deec0797aa1e685a7f8d78468122 /builtin-verify-pack.c
parent3449f8c4cb93a0ec445db22ee7549d0b051446d6 (diff)
downloadgit-340814636dde3cbd2e461b12f9ae832d2100766a.tar.gz
make verify_one_pack() a bit less wrong wrt packed_git structure
Simply freeing it is wrong. There are many things attached to this structure that are not cleaned up. In practice this doesn't matter much since this happens just before the program exits, but it is still a bit more "correct" to leak it implicitly rather than explicitly. And therefore it is also a good idea to register it with install_packed_git(). Not only might it have better chance of being properly cleaned up if such functionality is implemented for the general case, but some functions like init_revindex() expect all packed_git instances to be globally accessible. Signed-off-by: Nicolas Pitre <nico@cam.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-verify-pack.c')
-rw-r--r--builtin-verify-pack.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-verify-pack.c b/builtin-verify-pack.c
index 4e31c273f4..4958bbbf11 100644
--- a/builtin-verify-pack.c
+++ b/builtin-verify-pack.c
@@ -40,8 +40,8 @@ static int verify_one_pack(const char *path, int verbose)
if (!pack)
return error("packfile %s not found.", arg);
+ install_packed_git(pack);
err = verify_pack(pack, verbose);
- free(pack);
return err;
}