packfile: convert mark_bad_packed_object() to object_id

All callers have full object IDs, so pass them on instead of just their
hash member.

Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
René Scharfe 2021-09-11 22:40:33 +02:00 коммит произвёл Junio C Hamano
Родитель 893b563505
Коммит 751530de5d
3 изменённых файлов: 8 добавлений и 8 удалений

Просмотреть файл

@ -1616,7 +1616,7 @@ static int do_oid_object_info_extended(struct repository *r,
return 0; return 0;
rtype = packed_object_info(r, e.p, e.offset, oi); rtype = packed_object_info(r, e.p, e.offset, oi);
if (rtype < 0) { if (rtype < 0) {
mark_bad_packed_object(e.p, real->hash); mark_bad_packed_object(e.p, real);
return do_oid_object_info_extended(r, real, oi, 0); return do_oid_object_info_extended(r, real, oi, 0);
} else if (oi->whence == OI_PACKED) { } else if (oi->whence == OI_PACKED) {
oi->u.packed.offset = e.offset; oi->u.packed.offset = e.offset;

Просмотреть файл

@ -1161,17 +1161,17 @@ int unpack_object_header(struct packed_git *p,
return type; return type;
} }
void mark_bad_packed_object(struct packed_git *p, const unsigned char *sha1) void mark_bad_packed_object(struct packed_git *p, const struct object_id *oid)
{ {
unsigned i; unsigned i;
const unsigned hashsz = the_hash_algo->rawsz; const unsigned hashsz = the_hash_algo->rawsz;
for (i = 0; i < p->num_bad_objects; i++) for (i = 0; i < p->num_bad_objects; i++)
if (hasheq(sha1, p->bad_object_sha1 + hashsz * i)) if (hasheq(oid->hash, p->bad_object_sha1 + hashsz * i))
return; return;
p->bad_object_sha1 = xrealloc(p->bad_object_sha1, p->bad_object_sha1 = xrealloc(p->bad_object_sha1,
st_mult(GIT_MAX_RAWSZ, st_mult(GIT_MAX_RAWSZ,
st_add(p->num_bad_objects, 1))); st_add(p->num_bad_objects, 1)));
hashcpy(p->bad_object_sha1 + hashsz * p->num_bad_objects, sha1); hashcpy(p->bad_object_sha1 + hashsz * p->num_bad_objects, oid->hash);
p->num_bad_objects++; p->num_bad_objects++;
} }
@ -1272,7 +1272,7 @@ static int retry_bad_packed_offset(struct repository *r,
if (offset_to_pack_pos(p, obj_offset, &pos) < 0) if (offset_to_pack_pos(p, obj_offset, &pos) < 0)
return OBJ_BAD; return OBJ_BAD;
nth_packed_object_id(&oid, p, pack_pos_to_index(p, pos)); nth_packed_object_id(&oid, p, pack_pos_to_index(p, pos));
mark_bad_packed_object(p, oid.hash); mark_bad_packed_object(p, &oid);
type = oid_object_info(r, &oid, NULL); type = oid_object_info(r, &oid, NULL);
if (type <= OBJ_NONE) if (type <= OBJ_NONE)
return OBJ_BAD; return OBJ_BAD;
@ -1722,7 +1722,7 @@ void *unpack_entry(struct repository *r, struct packed_git *p, off_t obj_offset,
nth_packed_object_id(&oid, p, index_pos); nth_packed_object_id(&oid, p, index_pos);
error("bad packed object CRC for %s", error("bad packed object CRC for %s",
oid_to_hex(&oid)); oid_to_hex(&oid));
mark_bad_packed_object(p, oid.hash); mark_bad_packed_object(p, &oid);
data = NULL; data = NULL;
goto out; goto out;
} }
@ -1811,7 +1811,7 @@ void *unpack_entry(struct repository *r, struct packed_git *p, off_t obj_offset,
" at offset %"PRIuMAX" from %s", " at offset %"PRIuMAX" from %s",
oid_to_hex(&base_oid), (uintmax_t)obj_offset, oid_to_hex(&base_oid), (uintmax_t)obj_offset,
p->pack_name); p->pack_name);
mark_bad_packed_object(p, base_oid.hash); mark_bad_packed_object(p, &base_oid);
base = read_object(r, &base_oid, &type, &base_size); base = read_object(r, &base_oid, &type, &base_size);
external_base = base; external_base = base;
} }

Просмотреть файл

@ -159,7 +159,7 @@ int packed_object_info(struct repository *r,
struct packed_git *pack, struct packed_git *pack,
off_t offset, struct object_info *); off_t offset, struct object_info *);
void mark_bad_packed_object(struct packed_git *p, const unsigned char *sha1); void mark_bad_packed_object(struct packed_git *, const struct object_id *);
const struct packed_git *has_packed_and_bad(struct repository *r, const unsigned char *sha1); const struct packed_git *has_packed_and_bad(struct repository *r, const unsigned char *sha1);
#define ON_DISK_KEEP_PACKS 1 #define ON_DISK_KEEP_PACKS 1