From: Johannes Löthberg Date: Fri, 12 Jun 2015 20:15:15 GMT Subject: [PATCH/RFC] upload-pack: Fail if cloning empty namespace Message-ID: <1434140115-3139-1-git-send-email-johannes@kyriasis.com> Git should fail to clone if trying to clone from an non-existing ref namespace, since it's the same as a non-existing repository Signed-off-by: Johannes Löthberg --- In version 4 of the ArchLinux User Repository, which is a hosting platform for recepies for building Arch packages, we use Git to store the recepies. To save space we are using ref namespaces, which so far has saved quite a bit of space. There is one issue though, when cloning a non-existing repository we don't want the clone to work. This patch fixes this issue by failing the clone if a namespace was specified but it doesn't exist. Making this conditional on a namespace being specified makes sure that cloning regular empty repos still works. upload-pack.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/upload-pack.c b/upload-pack.c index 89e832b..21f8891 100644 --- a/upload-pack.c +++ b/upload-pack.c @@ -778,6 +778,10 @@ static void upload_pack(void) head_ref_namespaced(find_symref, &symref); + if (get_git_namespace() && !symref.items) { + die("git upload-pack: tried to clone from empty namespace"); + } + if (advertise_refs || !stateless_rpc) { reset_timeout(); head_ref_namespaced(send_ref, &symref); -- 2.4.2