mirror/ git
1
0
Fork 0
mirror of https://github.com/git/git.git synced 2024-05-29 00:26:12 +02:00
Code

Merge branch 'jt/upload-pack-v2-fix-shallow' 

"git fetch" over protocol v2 into a shallow repository failed to
fetch full history behind a new tip of history that was diverged
before the cut-off point of the history that was previously fetched
shallowly.

* jt/upload-pack-v2-fix-shallow:
  upload-pack: clear flags before each v2 request
  upload-pack: make want_obj not global
  upload-pack: make have_obj not global
This commit is contained in:
Junio C Hamano 2018-11-06 15:50:19 +09:00
parent 3fc8522e68 d1035cac09
commit 3df27e0e34
2 changed files with 115 additions and 63 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
t/t5702-protocol-v2.sh
View File

@ -446,6 +446,31 @@ test_expect_success 'fetch supports include-tag and tag following' '
git -C client cat-file -e $(git -C client rev-parse annotated_tag) git -C client cat-file -e $(git -C client rev-parse annotated_tag)
' '
test_expect_success 'upload-pack respects client shallows' '
rm -rf server client trace &&
git init server &&
test_commit -C server base &&
test_commit -C server client_has &&
git clone --depth=1 "file://$(pwd)/server" client &&
# Add extra commits to the client so that the whole fetch takes more
# than 1 request (due to negotiation)
for i in $(test_seq 1 32)
do
test_commit -C client c$i
done &&
git -C server checkout -b newbranch base &&
test_commit -C server client_wants &&
GIT_TRACE_PACKET="$(pwd)/trace" git -C client -c protocol.version=2 \
fetch origin newbranch &&
# Ensure that protocol v2 is used
grep "fetch< version 2" trace
'
# Test protocol v2 with 'http://' transport # Test protocol v2 with 'http://' transport
# #
. "$TEST_DIRECTORY"/lib-httpd.sh . "$TEST_DIRECTORY"/lib-httpd.sh

153
upload-pack.c
View File

@ -38,6 +38,9 @@
#define CLIENT_SHALLOW (1u << 18) #define CLIENT_SHALLOW (1u << 18)
#define HIDDEN_REF (1u << 19) #define HIDDEN_REF (1u << 19)
#define ALL_FLAGS (THEY_HAVE | OUR_REF | WANTED | COMMON_KNOWN | SHALLOW | \
NOT_SHALLOW | CLIENT_SHALLOW | HIDDEN_REF)
static timestamp_t oldest_have; static timestamp_t oldest_have;
static int deepen_relative; static int deepen_relative;
@ -53,8 +56,6 @@ static int no_progress, daemon_mode;
#define ALLOW_ANY_SHA1 07 #define ALLOW_ANY_SHA1 07
static unsigned int allow_unadvertised_object_request; static unsigned int allow_unadvertised_object_request;
static int shallow_nr; static int shallow_nr;
static struct object_array have_obj;
static struct object_array want_obj;
static struct object_array extra_edge_obj; static struct object_array extra_edge_obj;
static unsigned int timeout; static unsigned int timeout;
static int keepalive = 5; static int keepalive = 5;
@ -100,7 +101,8 @@ static int write_one_shallow(const struct commit_graft *graft, void *cb_data)
return 0; return 0;
} }
static void create_pack_file(void) static void create_pack_file(const struct object_array *have_obj,
const struct object_array *want_obj)
{ {
struct child_process pack_objects = CHILD_PROCESS_INIT; struct child_process pack_objects = CHILD_PROCESS_INIT;
char data[8193], progress[128]; char data[8193], progress[128];
@ -161,13 +163,13 @@ static void create_pack_file(void)
if (shallow_nr) if (shallow_nr)
for_each_commit_graft(write_one_shallow, pipe_fd); for_each_commit_graft(write_one_shallow, pipe_fd);
for (i = 0; i < want_obj.nr; i++) for (i = 0; i < want_obj->nr; i++)
fprintf(pipe_fd, "%s\n", fprintf(pipe_fd, "%s\n",
oid_to_hex(&want_obj.objects[i].item->oid)); oid_to_hex(&want_obj->objects[i].item->oid));
fprintf(pipe_fd, "--not\n"); fprintf(pipe_fd, "--not\n");
for (i = 0; i < have_obj.nr; i++) for (i = 0; i < have_obj->nr; i++)
fprintf(pipe_fd, "%s\n", fprintf(pipe_fd, "%s\n",
oid_to_hex(&have_obj.objects[i].item->oid)); oid_to_hex(&have_obj->objects[i].item->oid));
for (i = 0; i < extra_edge_obj.nr; i++) for (i = 0; i < extra_edge_obj.nr; i++)
fprintf(pipe_fd, "%s\n", fprintf(pipe_fd, "%s\n",
oid_to_hex(&extra_edge_obj.objects[i].item->oid)); oid_to_hex(&extra_edge_obj.objects[i].item->oid));
@ -304,7 +306,8 @@ static void create_pack_file(void)
die("git upload-pack: %s", abort_msg); die("git upload-pack: %s", abort_msg);
} }
static int got_oid(const char *hex, struct object_id *oid) static int got_oid(const char *hex, struct object_id *oid,
struct object_array *have_obj)
{ {
struct object *o; struct object *o;
int we_knew_they_have = 0; int we_knew_they_have = 0;
@ -332,25 +335,27 @@ static int got_oid(const char *hex, struct object_id *oid)
parents->item->object.flags |= THEY_HAVE; parents->item->object.flags |= THEY_HAVE;
} }
if (!we_knew_they_have) { if (!we_knew_they_have) {
add_object_array(o, NULL, &have_obj); add_object_array(o, NULL, have_obj);
return 1; return 1;
} }
return 0; return 0;
} }
static int ok_to_give_up(void) static int ok_to_give_up(const struct object_array *have_obj,
struct object_array *want_obj)
{ {
uint32_t min_generation = GENERATION_NUMBER_ZERO; uint32_t min_generation = GENERATION_NUMBER_ZERO;
if (!have_obj.nr) if (!have_obj->nr)
return 0; return 0;
return can_all_from_reach_with_flag(&want_obj, THEY_HAVE, return can_all_from_reach_with_flag(want_obj, THEY_HAVE,
COMMON_KNOWN, oldest_have, COMMON_KNOWN, oldest_have,
min_generation); min_generation);
} }
static int get_common_commits(void) static int get_common_commits(struct object_array *have_obj,
struct object_array *want_obj)
{ {
struct object_id oid; struct object_id oid;
char last_hex[GIT_MAX_HEXSZ + 1]; char last_hex[GIT_MAX_HEXSZ + 1];
@ -368,11 +373,11 @@ static int get_common_commits(void)
if (!line) { if (!line) {
if (multi_ack == 2 && got_common if (multi_ack == 2 && got_common
&& !got_other && ok_to_give_up()) { && !got_other && ok_to_give_up(have_obj, want_obj)) {
sent_ready = 1; sent_ready = 1;
packet_write_fmt(1, "ACK %s ready\n", last_hex); packet_write_fmt(1, "ACK %s ready\n", last_hex);
} }
if (have_obj.nr == 0 || multi_ack) if (have_obj->nr == 0 || multi_ack)
packet_write_fmt(1, "NAK\n"); packet_write_fmt(1, "NAK\n");
if (no_done && sent_ready) { if (no_done && sent_ready) {
@ -386,10 +391,10 @@ static int get_common_commits(void)
continue; continue;
} }
if (skip_prefix(line, "have ", &arg)) { if (skip_prefix(line, "have ", &arg)) {
switch (got_oid(arg, &oid)) { switch (got_oid(arg, &oid, have_obj)) {
case -1: /* they have what we do not */ case -1: /* they have what we do not */
got_other = 1; got_other = 1;
if (multi_ack && ok_to_give_up()) { if (multi_ack && ok_to_give_up(have_obj, want_obj)) {
const char *hex = oid_to_hex(&oid); const char *hex = oid_to_hex(&oid);
if (multi_ack == 2) { if (multi_ack == 2) {
sent_ready = 1; sent_ready = 1;
@ -405,14 +410,14 @@ static int get_common_commits(void)
packet_write_fmt(1, "ACK %s common\n", last_hex); packet_write_fmt(1, "ACK %s common\n", last_hex);
else if (multi_ack) else if (multi_ack)
packet_write_fmt(1, "ACK %s continue\n", last_hex); packet_write_fmt(1, "ACK %s continue\n", last_hex);
else if (have_obj.nr == 1) else if (have_obj->nr == 1)
packet_write_fmt(1, "ACK %s\n", last_hex); packet_write_fmt(1, "ACK %s\n", last_hex);
break; break;
} }
continue; continue;
} }
if (!strcmp(line, "done")) { if (!strcmp(line, "done")) {
if (have_obj.nr > 0) { if (have_obj->nr > 0) {
if (multi_ack) if (multi_ack)
packet_write_fmt(1, "ACK %s\n", last_hex); packet_write_fmt(1, "ACK %s\n", last_hex);
return 0; return 0;
@ -583,7 +588,7 @@ static int has_unreachable(struct object_array *src)
return 1; return 1;
} }
static void check_non_tip(void) static void check_non_tip(struct object_array *want_obj)
{ {
int i; int i;
@ -594,14 +599,14 @@ static void check_non_tip(void)
*/ */
if (!stateless_rpc && !(allow_unadvertised_object_request & ALLOW_REACHABLE_SHA1)) if (!stateless_rpc && !(allow_unadvertised_object_request & ALLOW_REACHABLE_SHA1))
goto error; goto error;
if (!has_unreachable(&want_obj)) if (!has_unreachable(want_obj))
/* All the non-tip ones are ancestors of what we advertised */ /* All the non-tip ones are ancestors of what we advertised */
return; return;
error: error:
/* Pick one of them (we know there at least is one) */ /* Pick one of them (we know there at least is one) */
for (i = 0; i < want_obj.nr; i++) { for (i = 0; i < want_obj->nr; i++) {
struct object *o = want_obj.objects[i].item; struct object *o = want_obj->objects[i].item;
if (!is_our_ref(o)) if (!is_our_ref(o))
die("git upload-pack: not our ref %s", die("git upload-pack: not our ref %s",
oid_to_hex(&o->oid)); oid_to_hex(&o->oid));
@ -622,7 +627,8 @@ static void send_shallow(struct commit_list *result)
} }
} }
static void send_unshallow(const struct object_array *shallows) static void send_unshallow(const struct object_array *shallows,
struct object_array *want_obj)
{ {
int i; int i;
@ -646,7 +652,7 @@ static void send_unshallow(const struct object_array *shallows)
parents = ((struct commit *)object)->parents; parents = ((struct commit *)object)->parents;
while (parents) { while (parents) {
add_object_array(&parents->item->object, add_object_array(&parents->item->object,
NULL, &want_obj); NULL, want_obj);
parents = parents->next; parents = parents->next;
} }
add_object_array(object, NULL, &extra_edge_obj); add_object_array(object, NULL, &extra_edge_obj);
@ -657,7 +663,7 @@ static void send_unshallow(const struct object_array *shallows)
} }
static void deepen(int depth, int deepen_relative, static void deepen(int depth, int deepen_relative,
struct object_array *shallows) struct object_array *shallows, struct object_array *want_obj)
{ {
if (depth == INFINITE_DEPTH && !is_repository_shallow(the_repository)) { if (depth == INFINITE_DEPTH && !is_repository_shallow(the_repository)) {
int i; int i;
@ -680,17 +686,18 @@ static void deepen(int depth, int deepen_relative,
} else { } else {
struct commit_list *result; struct commit_list *result;
result = get_shallow_commits(&want_obj, depth, result = get_shallow_commits(want_obj, depth,
SHALLOW, NOT_SHALLOW); SHALLOW, NOT_SHALLOW);
send_shallow(result); send_shallow(result);
free_commit_list(result); free_commit_list(result);
} }
send_unshallow(shallows); send_unshallow(shallows, want_obj);
} }
static void deepen_by_rev_list(int ac, const char **av, static void deepen_by_rev_list(int ac, const char **av,
struct object_array *shallows) struct object_array *shallows,
struct object_array *want_obj)
{ {
struct commit_list *result; struct commit_list *result;
@ -698,21 +705,22 @@ static void deepen_by_rev_list(int ac, const char **av,
result = get_shallow_commits_by_rev_list(ac, av, SHALLOW, NOT_SHALLOW); result = get_shallow_commits_by_rev_list(ac, av, SHALLOW, NOT_SHALLOW);
send_shallow(result); send_shallow(result);
free_commit_list(result); free_commit_list(result);
send_unshallow(shallows); send_unshallow(shallows, want_obj);
} }
/* Returns 1 if a shallow list is sent or 0 otherwise */ /* Returns 1 if a shallow list is sent or 0 otherwise */
static int send_shallow_list(int depth, int deepen_rev_list, static int send_shallow_list(int depth, int deepen_rev_list,
timestamp_t deepen_since, timestamp_t deepen_since,
struct string_list *deepen_not, struct string_list *deepen_not,
struct object_array *shallows) struct object_array *shallows,
struct object_array *want_obj)
{ {
int ret = 0; int ret = 0;
if (depth > 0 && deepen_rev_list) if (depth > 0 && deepen_rev_list)
die("git upload-pack: deepen and deepen-since (or deepen-not) cannot be used together"); die("git upload-pack: deepen and deepen-since (or deepen-not) cannot be used together");
if (depth > 0) { if (depth > 0) {
deepen(depth, deepen_relative, shallows); deepen(depth, deepen_relative, shallows, want_obj);
ret = 1; ret = 1;
} else if (deepen_rev_list) { } else if (deepen_rev_list) {
struct argv_array av = ARGV_ARRAY_INIT; struct argv_array av = ARGV_ARRAY_INIT;
@ -729,11 +737,11 @@ static int send_shallow_list(int depth, int deepen_rev_list,
} }
argv_array_push(&av, "--not"); argv_array_push(&av, "--not");
} }
for (i = 0; i < want_obj.nr; i++) { for (i = 0; i < want_obj->nr; i++) {
struct object *o = want_obj.objects[i].item; struct object *o = want_obj->objects[i].item;
argv_array_push(&av, oid_to_hex(&o->oid)); argv_array_push(&av, oid_to_hex(&o->oid));
} }
deepen_by_rev_list(av.argc, av.argv, shallows); deepen_by_rev_list(av.argc, av.argv, shallows, want_obj);
argv_array_clear(&av); argv_array_clear(&av);
ret = 1; ret = 1;
} else { } else {
@ -818,7 +826,7 @@ static int process_deepen_not(const char *line, struct string_list *deepen_not,
return 0; return 0;
} }
static void receive_needs(void) static void receive_needs(struct object_array *want_obj)
{ {
struct object_array shallows = OBJECT_ARRAY_INIT; struct object_array shallows = OBJECT_ARRAY_INIT;
struct string_list deepen_not = STRING_LIST_INIT_DUP; struct string_list deepen_not = STRING_LIST_INIT_DUP;
@ -896,7 +904,7 @@ static void receive_needs(void)
if (!((allow_unadvertised_object_request & ALLOW_ANY_SHA1) == ALLOW_ANY_SHA1 if (!((allow_unadvertised_object_request & ALLOW_ANY_SHA1) == ALLOW_ANY_SHA1
|| is_our_ref(o))) || is_our_ref(o)))
has_non_tip = 1; has_non_tip = 1;
add_object_array(o, NULL, &want_obj); add_object_array(o, NULL, want_obj);
} }
} }
@ -908,7 +916,7 @@ static void receive_needs(void)
* by another process that handled the initial request. * by another process that handled the initial request.
*/ */
if (has_non_tip) if (has_non_tip)
check_non_tip(); check_non_tip(want_obj);
if (!use_sideband && daemon_mode) if (!use_sideband && daemon_mode)
no_progress = 1; no_progress = 1;
@ -917,7 +925,7 @@ static void receive_needs(void)
return; return;
if (send_shallow_list(depth, deepen_rev_list, deepen_since, if (send_shallow_list(depth, deepen_rev_list, deepen_since,
&deepen_not, &shallows)) &deepen_not, &shallows, want_obj))
packet_flush(1); packet_flush(1);
object_array_clear(&shallows); object_array_clear(&shallows);
} }
@ -1046,6 +1054,7 @@ static int upload_pack_config(const char *var, const char *value, void *unused)
void upload_pack(struct upload_pack_options *options) void upload_pack(struct upload_pack_options *options)
{ {
struct string_list symref = STRING_LIST_INIT_DUP; struct string_list symref = STRING_LIST_INIT_DUP;
struct object_array want_obj = OBJECT_ARRAY_INIT;
stateless_rpc = options->stateless_rpc; stateless_rpc = options->stateless_rpc;
timeout = options->timeout; timeout = options->timeout;
@ -1069,10 +1078,11 @@ void upload_pack(struct upload_pack_options *options)
if (options->advertise_refs) if (options->advertise_refs)
return; return;
receive_needs(); receive_needs(&want_obj);
if (want_obj.nr) { if (want_obj.nr) {
get_common_commits(); struct object_array have_obj = OBJECT_ARRAY_INIT;
create_pack_file(); get_common_commits(&have_obj, &want_obj);
create_pack_file(&have_obj, &want_obj);
} }
} }
@ -1122,7 +1132,7 @@ static void upload_pack_data_clear(struct upload_pack_data *data)
string_list_clear(&data->deepen_not, 0); string_list_clear(&data->deepen_not, 0);
} }
static int parse_want(const char *line) static int parse_want(const char *line, struct object_array *want_obj)
{ {
const char *arg; const char *arg;
if (skip_prefix(line, "want ", &arg)) { if (skip_prefix(line, "want ", &arg)) {
@ -1144,7 +1154,7 @@ static int parse_want(const char *line)
if (!(o->flags & WANTED)) { if (!(o->flags & WANTED)) {
o->flags |= WANTED; o->flags |= WANTED;
add_object_array(o, NULL, &want_obj); add_object_array(o, NULL, want_obj);
} }
return 1; return 1;
@ -1153,7 +1163,8 @@ static int parse_want(const char *line)
return 0; return 0;
} }
static int parse_want_ref(const char *line, struct string_list *wanted_refs) static int parse_want_ref(const char *line, struct string_list *wanted_refs,
struct object_array *want_obj)
{ {
const char *arg; const char *arg;
if (skip_prefix(line, "want-ref ", &arg)) { if (skip_prefix(line, "want-ref ", &arg)) {
@ -1172,7 +1183,7 @@ static int parse_want_ref(const char *line, struct string_list *wanted_refs)
o = parse_object_or_die(&oid, arg); o = parse_object_or_die(&oid, arg);
if (!(o->flags & WANTED)) { if (!(o->flags & WANTED)) {
o->flags |= WANTED; o->flags |= WANTED;
add_object_array(o, NULL, &want_obj); add_object_array(o, NULL, want_obj);
} }
return 1; return 1;
@ -1197,16 +1208,18 @@ static int parse_have(const char *line, struct oid_array *haves)
} }
static void process_args(struct packet_reader *request, static void process_args(struct packet_reader *request,
struct upload_pack_data *data) struct upload_pack_data *data,
struct object_array *want_obj)
{ {
while (packet_reader_read(request) != PACKET_READ_FLUSH) { while (packet_reader_read(request) != PACKET_READ_FLUSH) {
const char *arg = request->line; const char *arg = request->line;
const char *p; const char *p;
/* process want */ /* process want */
if (parse_want(arg)) if (parse_want(arg, want_obj))
continue; continue;
if (allow_ref_in_want && parse_want_ref(arg, &data->wanted_refs)) if (allow_ref_in_want &&
parse_want_ref(arg, &data->wanted_refs, want_obj))
continue; continue;
/* process have line */ /* process have line */
if (parse_have(arg, &data->haves)) if (parse_have(arg, &data->haves))
@ -1260,7 +1273,8 @@ static void process_args(struct packet_reader *request,
} }
} }
static int process_haves(struct oid_array *haves, struct oid_array *common) static int process_haves(struct oid_array *haves, struct oid_array *common,
struct object_array *have_obj)
{ {
int i; int i;
@ -1293,13 +1307,15 @@ static int process_haves(struct oid_array *haves, struct oid_array *common)
parents->item->object.flags |= THEY_HAVE; parents->item->object.flags |= THEY_HAVE;
} }
if (!we_knew_they_have) if (!we_knew_they_have)
add_object_array(o, NULL, &have_obj); add_object_array(o, NULL, have_obj);
} }
return 0; return 0;
} }
static int send_acks(struct oid_array *acks, struct strbuf *response) static int send_acks(struct oid_array *acks, struct strbuf *response,
const struct object_array *have_obj,
struct object_array *want_obj)
{ {
int i; int i;
@ -1314,7 +1330,7 @@ static int send_acks(struct oid_array *acks, struct strbuf *response)
oid_to_hex(&acks->oid[i])); oid_to_hex(&acks->oid[i]));
} }
if (ok_to_give_up()) { if (ok_to_give_up(have_obj, want_obj)) {
/* Send Ready */ /* Send Ready */
packet_buf_write(response, "ready\n"); packet_buf_write(response, "ready\n");
return 1; return 1;
@ -1323,16 +1339,18 @@ static int send_acks(struct oid_array *acks, struct strbuf *response)
return 0; return 0;
} }
static int process_haves_and_send_acks(struct upload_pack_data *data) static int process_haves_and_send_acks(struct upload_pack_data *data,
struct object_array *have_obj,
struct object_array *want_obj)
{ {
struct oid_array common = OID_ARRAY_INIT; struct oid_array common = OID_ARRAY_INIT;
struct strbuf response = STRBUF_INIT; struct strbuf response = STRBUF_INIT;
int ret = 0; int ret = 0;
process_haves(&data->haves, &common); process_haves(&data->haves, &common, have_obj);
if (data->done) { if (data->done) {
ret = 1; ret = 1;
} else if (send_acks(&common, &response)) { } else if (send_acks(&common, &response, have_obj, want_obj)) {
packet_buf_delim(&response); packet_buf_delim(&response);
ret = 1; ret = 1;
} else { } else {
@ -1368,7 +1386,8 @@ static void send_wanted_ref_info(struct upload_pack_data *data)
packet_delim(1); packet_delim(1);
} }
static void send_shallow_info(struct upload_pack_data *data) static void send_shallow_info(struct upload_pack_data *data,
struct object_array *want_obj)
{ {
/* No shallow info needs to be sent */ /* No shallow info needs to be sent */
if (!data->depth && !data->deepen_rev_list && !data->shallows.nr && if (!data->depth && !data->deepen_rev_list && !data->shallows.nr &&
@ -1379,9 +1398,10 @@ static void send_shallow_info(struct upload_pack_data *data)
if (!send_shallow_list(data->depth, data->deepen_rev_list, if (!send_shallow_list(data->depth, data->deepen_rev_list,
data->deepen_since, &data->deepen_not, data->deepen_since, &data->deepen_not,
&data->shallows) && &data->shallows, want_obj) &&
is_repository_shallow(the_repository)) is_repository_shallow(the_repository))
deepen(INFINITE_DEPTH, data->deepen_relative, &data->shallows); deepen(INFINITE_DEPTH, data->deepen_relative, &data->shallows,
want_obj);
packet_delim(1); packet_delim(1);
} }
@ -1398,6 +1418,10 @@ int upload_pack_v2(struct repository *r, struct argv_array *keys,
{ {
enum fetch_state state = FETCH_PROCESS_ARGS; enum fetch_state state = FETCH_PROCESS_ARGS;
struct upload_pack_data data; struct upload_pack_data data;
struct object_array have_obj = OBJECT_ARRAY_INIT;
struct object_array want_obj = OBJECT_ARRAY_INIT;
clear_object_flags(ALL_FLAGS);
git_config(upload_pack_config, NULL); git_config(upload_pack_config, NULL);
@ -1407,7 +1431,7 @@ int upload_pack_v2(struct repository *r, struct argv_array *keys,
while (state != FETCH_DONE) { while (state != FETCH_DONE) {
switch (state) { switch (state) {
case FETCH_PROCESS_ARGS: case FETCH_PROCESS_ARGS:
process_args(request, &data); process_args(request, &data, &want_obj);
if (!want_obj.nr) { if (!want_obj.nr) {
/* /*
@ -1429,17 +1453,18 @@ int upload_pack_v2(struct repository *r, struct argv_array *keys,
} }
break; break;
case FETCH_SEND_ACKS: case FETCH_SEND_ACKS:
if (process_haves_and_send_acks(&data)) if (process_haves_and_send_acks(&data, &have_obj,
&want_obj))
state = FETCH_SEND_PACK; state = FETCH_SEND_PACK;
else else
state = FETCH_DONE; state = FETCH_DONE;
break; break;
case FETCH_SEND_PACK: case FETCH_SEND_PACK:
send_wanted_ref_info(&data); send_wanted_ref_info(&data);
send_shallow_info(&data); send_shallow_info(&data, &want_obj);
packet_write_fmt(1, "packfile\n"); packet_write_fmt(1, "packfile\n");
create_pack_file(); create_pack_file(&have_obj, &want_obj);
state = FETCH_DONE; state = FETCH_DONE;
break; break;
case FETCH_DONE: case FETCH_DONE:
@ -1448,6 +1473,8 @@ int upload_pack_v2(struct repository *r, struct argv_array *keys,
} }
upload_pack_data_clear(&data); upload_pack_data_clear(&data);
object_array_clear(&have_obj);
object_array_clear(&want_obj);
return 0; return 0;
} }