Commit aeba5e175a58eebc5eae43adaa40955d2a9f55c3

Carlos Martín Nieto 2012-10-10T23:55:03

http: don't discard the HEAD ref The fix for fetching from empty repositories (22935b06d protocol: don't store flushes; 2012-10-07) forgot to take into account the deletion of the flush pkt in the HTTP transport. As a result, the HEAD ref advertisement where we detect the remote's capabilities was deleted instead. Fix this.

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
diff --git a/src/transports/http.c b/src/transports/http.c
index d5015f5..7b77c92 100644
--- a/src/transports/http.c
+++ b/src/transports/http.c
@@ -473,10 +473,7 @@ static int http_connect(git_transport *transport, int direction)
 		giterr_set(GITERR_NET, "Invalid HTTP response");
 		return t->error = -1;
 	} else {
-		/* Remove the comment and flush pkts */
-		git_vector_remove(&transport->refs, 0);
-		git__free(pkt);
-		pkt = git_vector_get(&transport->refs, 0);
+		/* Remove the comment pkt from the list */
 		git_vector_remove(&transport->refs, 0);
 		git__free(pkt);
 	}