From a096bb18af5b7214d9b87b64ae733d106eb96544 Mon Sep 17 00:00:00 2001 From: Mike Hommey Date: Fri, 14 Dec 2007 22:18:01 +0100 Subject: Fix random sha1 in error message in http-fetch and http-push When a downloaded ref doesn't contain a sha1, the error message displays a random sha1 because of uninitialized memory. This happens when cloning a repository that is already a clone of another one, in which case refs/remotes/origin/HEAD is a symref. Signed-off-by: Mike Hommey Signed-off-by: Junio C Hamano --- http-walker.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'http-walker.c') diff --git a/http-walker.c b/http-walker.c index a3fb596542..0faafbce61 100644 --- a/http-walker.c +++ b/http-walker.c @@ -986,9 +986,10 @@ static int fetch_ref(struct walker *walker, char *ref, unsigned char *sha1) return error("Unable to start request"); } + if (buffer.posn != 41) + return 1; hex[40] = '\0'; - get_sha1_hex(hex, sha1); - return 0; + return get_sha1_hex(hex, sha1); } static void cleanup(struct walker *walker) -- cgit v1.2.1