mirror of
https://github.com/git/git
synced 2024-08-25 02:35:46 +00:00
![Nicolas Pitre](/assets/img/avatar_default.png)
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>
81 lines
1.7 KiB
C
81 lines
1.7 KiB
C
#include "builtin.h"
|
|
#include "cache.h"
|
|
#include "pack.h"
|
|
|
|
static int verify_one_pack(const char *path, int verbose)
|
|
{
|
|
char arg[PATH_MAX];
|
|
int len;
|
|
struct packed_git *pack;
|
|
int err;
|
|
|
|
len = strlcpy(arg, path, PATH_MAX);
|
|
if (len >= PATH_MAX)
|
|
return error("name too long: %s", path);
|
|
|
|
/*
|
|
* In addition to "foo.idx" we accept "foo.pack" and "foo";
|
|
* normalize these forms to "foo.idx" for add_packed_git().
|
|
*/
|
|
if (has_extension(arg, ".pack")) {
|
|
strcpy(arg + len - 5, ".idx");
|
|
len--;
|
|
} else if (!has_extension(arg, ".idx")) {
|
|
if (len + 4 >= PATH_MAX)
|
|
return error("name too long: %s.idx", arg);
|
|
strcpy(arg + len, ".idx");
|
|
len += 4;
|
|
}
|
|
|
|
/*
|
|
* add_packed_git() uses our buffer (containing "foo.idx") to
|
|
* build the pack filename ("foo.pack"). Make sure it fits.
|
|
*/
|
|
if (len + 1 >= PATH_MAX) {
|
|
arg[len - 4] = '\0';
|
|
return error("name too long: %s.pack", arg);
|
|
}
|
|
|
|
pack = add_packed_git(arg, len, 1);
|
|
if (!pack)
|
|
return error("packfile %s not found.", arg);
|
|
|
|
install_packed_git(pack);
|
|
err = verify_pack(pack, verbose);
|
|
|
|
return err;
|
|
}
|
|
|
|
static const char verify_pack_usage[] = "git-verify-pack [-v] <pack>...";
|
|
|
|
int cmd_verify_pack(int argc, const char **argv, const char *prefix)
|
|
{
|
|
int err = 0;
|
|
int verbose = 0;
|
|
int no_more_options = 0;
|
|
int nothing_done = 1;
|
|
|
|
git_config(git_default_config);
|
|
while (1 < argc) {
|
|
if (!no_more_options && argv[1][0] == '-') {
|
|
if (!strcmp("-v", argv[1]))
|
|
verbose = 1;
|
|
else if (!strcmp("--", argv[1]))
|
|
no_more_options = 1;
|
|
else
|
|
usage(verify_pack_usage);
|
|
}
|
|
else {
|
|
if (verify_one_pack(argv[1], verbose))
|
|
err = 1;
|
|
nothing_done = 0;
|
|
}
|
|
argc--; argv++;
|
|
}
|
|
|
|
if (nothing_done)
|
|
usage(verify_pack_usage);
|
|
|
|
return err;
|
|
}
|