]> Pileus Git - ~andy/git/commitdiff
upload-pack: make sure "want" objects are parsed
authorJeff King <peff@peff.net>
Sat, 16 Mar 2013 10:27:01 +0000 (06:27 -0400)
committerJunio C Hamano <gitster@pobox.com>
Sun, 17 Mar 2013 05:16:56 +0000 (22:16 -0700)
When upload-pack receives a "want" line from the client, it
adds it to an object array. We call lookup_object to find
the actual object, which will only check for objects already
in memory. This works because we are expecting to find
objects that we already loaded during the ref advertisement.

We use the resulting object structs for a variety of
purposes. Some of them care only about the object flags, but
others care about the type of the object (e.g.,
ok_to_give_up), or even feed them to the revision parser
(when --depth is used), which assumes that objects it
receives are fully parsed.

Once upon a time, this was OK; any object we loaded into
memory would also have been parsed. But since 435c833
(upload-pack: use peel_ref for ref advertisements,
2012-10-04), we try to avoid parsing objects during the ref
advertisement. This means that lookup_object may return an
object with a type of OBJ_NONE. The resulting mess depends
on the exact set of objects, but can include the revision
parser barfing, or the shallow code sending the wrong set of
objects.

This patch teaches upload-pack to parse each "want" object
as we receive it. We do not replace the lookup_object call
with parse_object, as the current code is careful not to let
just any object appear on a "want" line, but rather only one
we have previously advertised (whereas parse_object would
actually load any arbitrary object from disk).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t5500-fetch-pack.sh
upload-pack.c

index e80a2af348565a0a3d001ef28cb76427acfee495..e202383e4f289bfb78ce2a960c522cc4c04103d8 100755 (executable)
@@ -331,6 +331,15 @@ EOF
        test_cmp count7.expected count7.actual
 '
 
+test_expect_success 'clone shallow with packed refs' '
+       git pack-refs --all &&
+       git clone --depth 1 --branch A "file://$(pwd)/." shallow8 &&
+       echo "in-pack: 4" > count8.expected &&
+       GIT_DIR=shallow8/.git git count-objects -v |
+               grep "^in-pack" > count8.actual &&
+       test_cmp count8.expected count8.actual
+'
+
 test_expect_success 'setup tests for the --stdin parameter' '
        for head in C D E F
        do
index e29d5d2085657a201f1d0028144e4cddaf64c611..b6ec605b0931314deddbbb4aa7ea553f495fc1cf 100644 (file)
@@ -639,7 +639,7 @@ static void receive_needs(void)
                        use_include_tag = 1;
 
                o = lookup_object(sha1_buf);
-               if (!o)
+               if (!o || !parse_object(o->sha1))
                        die("git upload-pack: not our ref %s",
                            sha1_to_hex(sha1_buf));
                if (!(o->flags & WANTED)) {