From patchwork Fri Mar 14 14:12:01 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christian Couder X-Patchwork-Id: 14016937 Received: from mail-wm1-f51.google.com (mail-wm1-f51.google.com [209.85.128.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EDF542747B for ; Fri, 14 Mar 2025 14:12:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741961539; cv=none; b=ere1XoHEHyndJmti+XZ5ylh2MrUF1gHXZhmvtcdu1x8NneeSmffGfkjVi35HWOG6isyiVlvCaWIQ1L2uEMDYTKddoVe85XMOCoe3jL8AKDqcE4mSMiAMw4nsfAC1RRei88TqhsJcCl9mjco1mRomtprHReGIUz2XcLmX+kbgals= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741961539; c=relaxed/simple; bh=QkSyPBkHkjJB+4Kv2c+wDkH//HIR9ppU/ox6QPxK/oc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=M+N3lmMPvNHns7jJL9p8w89BS8QpAHgMYSkfD3PWKT16b+CvPJmY8JHpvJgEvqCu79HRIREdOfkpDqrCA0RL6k4BONUmhk5RS61n/TgOKDfEYRyb7oGHxGohDWKHSzKzXfJrgxUXyOlqm/Titfs6TVsgcZsttuDIgLJOwjKxP2k= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=OUFR1eTS; arc=none smtp.client-ip=209.85.128.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="OUFR1eTS" Received: by mail-wm1-f51.google.com with SMTP id 5b1f17b1804b1-43cf848528aso20471395e9.2 for ; Fri, 14 Mar 2025 07:12:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1741961534; x=1742566334; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=buxXoi39BsewLOH6ecsBMyOTREdrfvKHNvSiCqSJFME=; b=OUFR1eTSOdindnZd+6dKREKRwDP6d1b+Y8sCTAMGp5vZXkMk8BDjL7GqvBYvEA0Tez rhIGaHMvLDTmCKPMwtTJd9Gm7vyNn8Mv2HzgszzysIZVHyX46AhrP+6aJJ50lB83bnOX ZATWrahe85uCn0W4E+VWEhK44SnzooLsFNLQ4MjQu4HD8zmrJ0dNE/XSG3aGZ5pkewH+ +KShV1OIV58Wd1SX/+Hfd3yRdhQntzRzK1WkAOw9rd+rhLn0t5G+uXY+1BjwU0/GHLzT MUIIO9HAOEr4ac9CnTPTebJduNNNmn4T82gakZTx5WLm2GO3aHd/dlCZuWQ6jJlNNt5/ lzrQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741961534; x=1742566334; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=buxXoi39BsewLOH6ecsBMyOTREdrfvKHNvSiCqSJFME=; b=c1qCKzPIqjP+G/250LLQxH04HuIP/G5XG+5wfOvOjYCj7FvDe9KZEn8pRvAFab8IFV kluTsH0Vd745XLzhmNwu7nwhr+v7Hi7zHrjIXsojhaxLmF2/S6O7S5Vzd7MCZ4I5BYui U3s8n1bvh9ndEaTwrVNaEbM1t7RwKd2kOm7urRhFTIW2VynB+NIWti49nZuwBXBfh4VB UWazSpZounx3Kweh0IiR4dC1dVZ58L9MthZROdm92ayBQcuPm/RV9Q8LSAvElCJkEgbg 3q0wPMqagZyrhiYc/E8MSDCRyc4vdSnC7pr2jldFdSv79dSPAjutM4imlMkIQgqatRjK Xu0g== X-Gm-Message-State: AOJu0YyQc2ar7F/j8qpPBRjCVqDBTEGgwk4vWr3BJILz2ilT0HnRNn0L 7JA7sUNT46p9ON9DOgGZpPHqJhGcUtlbvSyHHn/Fksdza15KoRKwVWsHOw== X-Gm-Gg: ASbGnctJIkOmUVdrpdD0MM5IPfBMlIUThk3e7EuHI2SaKHIU2gsfzSAbHyYvghE/Yg7 5j3GjgBqGyt4e5MWu3bXwDwcur4GLMH+raWUGjFDFIYhtWMQAkDek/o/O8I5ANbIR3HBkpL0iuA l3AYndrrns4w5xXIHqzK5XyxsX32f0RCykU0AYUbY/Y/k1CAstqVxjyLerKWeStmUFZwUs+hXN+ agaxY9CmFqWxVd42JQo+RkgYfEWdRK6nmHsDkw8ReRs1bR3EeDS0a0tFrECj5gaWDz3xQKd7Tf6 gih6spnuSJ+inCotihyJ7gidtuPXQCyfsQUpndLI9vcmhWU+RXYt8BZluF26z52b69k3Sv+mBdC mqpWg0e7sh7JzgyZnXf4mQYhENpk= X-Google-Smtp-Source: AGHT+IGF+sQKrum0Xeo+aqiIDY1892W8NyR3ZY+N/V7/dQM7cv800eNnv3cNBeiAPeANZ2QbJ3NFUQ== X-Received: by 2002:a05:600c:1c20:b0:43c:fb8e:aec0 with SMTP id 5b1f17b1804b1-43d1ec72879mr25968045e9.1.1741961534228; Fri, 14 Mar 2025 07:12:14 -0700 (PDT) Received: from christian-Precision-5550.. (176-138-135-207.abo.bbox.fr. [176.138.135.207]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43d1fe3a8cdsm8418255e9.2.2025.03.14.07.12.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 14 Mar 2025 07:12:13 -0700 (PDT) From: Christian Couder To: git@vger.kernel.org Cc: Junio C Hamano , Jeff King , Patrick Steinhardt , Taylor Blau , Eric Sunshine , Karthik Nayak , Kristoffer Haugsbakk , "brian m . carlson" , "Randall S . Becker" , Christian Couder , Christian Couder Subject: [PATCH v5 1/3] promisor-remote: fix segfault when remote URL is missing Date: Fri, 14 Mar 2025 15:12:01 +0100 Message-ID: <20250314141203.2548803-2-christian.couder@gmail.com> X-Mailer: git-send-email 2.49.0.rc2.36.ga8a9f9b33b In-Reply-To: <20250314141203.2548803-1-christian.couder@gmail.com> References: <20250313103859.817127-1-christian.couder@gmail.com> <20250314141203.2548803-1-christian.couder@gmail.com> Precedence: bulk X-Mailing-List: git@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Using strvec_push() to push `NULL` into a 'strvec' results in a segfault, because `xstrdup(NULL)` crashes. So when an URL is missing from the config, let's not push the remote name and URL into the 'strvec's. While at it, let's also not push them in case the URL is empty. It's just not worth the trouble and it's consistent with how Git otherwise treats missing and empty URLs in the same way. Note that in case of missing or empty URL, Git uses the remote name to fetch, which can work if the remote is on the same filesystem. So configurations where the client, server and remote are all on the same filesystem may need URLs to be configured even if they are the same as the remote names. But this is a rare case, and the work around is easy enough. We leave improving the strvec API and/or xstrdup() for a future separate effort. While at it, let's also use git_config_get_string_tmp() instead of git_config_get_string() to simplify memory management. Helped-by: Jeff King Signed-off-by: Christian Couder --- promisor-remote.c | 16 ++++---- t/t5710-promisor-remote-capability.sh | 59 +++++++++++++++++++++++++++ 2 files changed, 67 insertions(+), 8 deletions(-) diff --git a/promisor-remote.c b/promisor-remote.c index 6a0a61382f..ba80240f12 100644 --- a/promisor-remote.c +++ b/promisor-remote.c @@ -323,13 +323,15 @@ static void promisor_info_vecs(struct repository *repo, promisor_remote_init(repo); for (r = repo->promisor_remote_config->promisors; r; r = r->next) { - char *url; + const char *url; char *url_key = xstrfmt("remote.%s.url", r->name); - strvec_push(names, r->name); - strvec_push(urls, git_config_get_string(url_key, &url) ? NULL : url); + /* Only add remotes with a non empty URL */ + if (!git_config_get_string_tmp(url_key, &url) && *url) { + strvec_push(names, r->name); + strvec_push(urls, url); + } - free(url); free(url_key); } } @@ -356,10 +358,8 @@ char *promisor_remote_info(struct repository *repo) strbuf_addch(&sb, ';'); strbuf_addstr(&sb, "name="); strbuf_addstr_urlencode(&sb, names.v[i], allow_unsanitized); - if (urls.v[i]) { - strbuf_addstr(&sb, ",url="); - strbuf_addstr_urlencode(&sb, urls.v[i], allow_unsanitized); - } + strbuf_addstr(&sb, ",url="); + strbuf_addstr_urlencode(&sb, urls.v[i], allow_unsanitized); } strvec_clear(&names); diff --git a/t/t5710-promisor-remote-capability.sh b/t/t5710-promisor-remote-capability.sh index d2cc69a17e..05ae96d1f7 100755 --- a/t/t5710-promisor-remote-capability.sh +++ b/t/t5710-promisor-remote-capability.sh @@ -193,6 +193,25 @@ test_expect_success "clone with 'KnownName' and different remote names" ' initialize_server 1 "$oid" ' +test_expect_success "clone with 'KnownName' and missing URL in the config" ' + git -C server config promisor.advertise true && + + # Clone from server to create a client + # Lazy fetching by the client from the LOP will fail because of the + # missing URL in the client config, so the server will have to lazy + # fetch from the LOP. + GIT_NO_LAZY_FETCH=0 git clone -c remote.lop.promisor=true \ + -c promisor.acceptfromserver=KnownName \ + --no-local --filter="blob:limit=5k" server client && + test_when_finished "rm -rf client" && + + # Check that the largest object is not missing on the server + check_missing_objects server 0 "" && + + # Reinitialize server so that the largest object is missing again + initialize_server 1 "$oid" +' + test_expect_success "clone with promisor.acceptfromserver set to 'KnownUrl'" ' git -C server config promisor.advertise true && @@ -228,6 +247,46 @@ test_expect_success "clone with 'KnownUrl' and different remote urls" ' initialize_server 1 "$oid" ' +test_expect_success "clone with 'KnownUrl' and url not configured on the server" ' + git -C server config promisor.advertise true && + + git -C server config unset remote.lop.url && + test_when_finished "git -C server config set remote.lop.url \"file://$(pwd)/lop\"" && + + # Clone from server to create a client + # It should fail because the client will reject the LOP as URLs are + # different, and the server cannot lazy fetch as the LOP URL is + # missing, so the remote name will be used instead which will fail. + test_must_fail env GIT_NO_LAZY_FETCH=0 git clone -c remote.lop.promisor=true \ + -c remote.lop.fetch="+refs/heads/*:refs/remotes/lop/*" \ + -c remote.lop.url="file://$(pwd)/lop" \ + -c promisor.acceptfromserver=KnownUrl \ + --no-local --filter="blob:limit=5k" server client && + + # Check that the largest object is still missing on the server + check_missing_objects server 1 "$oid" +' + +test_expect_success "clone with 'KnownUrl' and empty url, so not advertised" ' + git -C server config promisor.advertise true && + + git -C server config set remote.lop.url "" && + test_when_finished "git -C server config set remote.lop.url \"file://$(pwd)/lop\"" && + + # Clone from server to create a client + # It should fail because the client will reject the LOP as an empty URL is + # not advertised, and the server cannot lazy fetch as the LOP URL is empty, + # so the remote name will be used instead which will fail. + test_must_fail env GIT_NO_LAZY_FETCH=0 git clone -c remote.lop.promisor=true \ + -c remote.lop.fetch="+refs/heads/*:refs/remotes/lop/*" \ + -c remote.lop.url="file://$(pwd)/lop" \ + -c promisor.acceptfromserver=KnownUrl \ + --no-local --filter="blob:limit=5k" server client && + + # Check that the largest object is still missing on the server + check_missing_objects server 1 "$oid" +' + test_expect_success "clone with promisor.advertise set to 'true' but don't delete the client" ' git -C server config promisor.advertise true && From patchwork Fri Mar 14 14:12:02 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christian Couder X-Patchwork-Id: 14016938 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 863EC1FF7C1 for ; Fri, 14 Mar 2025 14:12:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741961540; cv=none; b=HkotMvtCzSN46+9i1pMY2vB5EpPiqjszhKmwSa9czf8ueGotWt2Uv+pdvp7sOehFVT0ZHf4DgjCfuS7dvvNKpSWr7KJ/gCtCMleaLof9G9QODHtGOVo4NaXu825bFJjV/0gyDbpT8nUIal9sQXEonjgGqJW7GRwd0ybcrwvqPVs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741961540; c=relaxed/simple; bh=laHD08UXkC6NYfki+PUjtH9iHHnBNo9I9+zBDCsujXk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=W+yfxQ7/QX7/9BG8eUx14bS6DcnVnxfkZbM0ZHV1Sae+s/t+2gCJ4ecjZuBOq26iDWtThyuzJzYb0/IfvyIST8QFw2w1cRgytSiakUzY/5/gcUmWjcmwwqMKT4YIL7v+fvoDVM0bSF9tgMJNhJLh9IYhsLfPmVqe19bkmhipBnI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Ix5eb+4s; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Ix5eb+4s" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-43cfa7e7f54so14375855e9.1 for ; Fri, 14 Mar 2025 07:12:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1741961535; x=1742566335; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=yZTvsrkNonguMWqOl7a5ms4H1yEv81A5pDVLgjYd46A=; b=Ix5eb+4saOyJP2le0L8ylesHqnICLhtSCTgFi4N5I1zqJSOeRtL2jAy5gOjdevgDzE vdDMRm2uPTeE5eYtJzoH2wLs3M/gkepfHnJc0SieaLV8HGuleuLvY+7/uPCGWuOdMwU1 w31kSETGEsepSWn2ynPM0eYeCW9NhL9mwFYAyyZ624lqM8Fc2I5ZBqBumAGZhwMl7vKr 04eN96lmIRzZHBiAJXuyo8LklgeMD7FNw1EU+UIlSOyAB4yY/rei95lgzuUjxGFLpE4U 2c1Pn1z9J8FxHiMsatq+a8c6kv7nYtyQPaW4+B6BFGEDGYbeZpqxiKFrK7zbcBdlwHS9 WFDQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741961535; x=1742566335; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=yZTvsrkNonguMWqOl7a5ms4H1yEv81A5pDVLgjYd46A=; b=enbUOwjdJB1XnOjo8AToW5zGt6kzsiDE43145gROdN/SkgS5VisFYqQ7AbaOT1exDW TLsGTrRsqYZSlB933gan3tLCNFgQHCzQljPFsbES13qmM8aBZDB9I3sOY2e2y/GprXSw bpuGQzgigOr1Rr8OoYuqBxuggHVYJRZc2WIT9xFHIxeqXIBWz4e00FHB+57tq9tIUfa1 QjJC8stdBf2Z+qMWLtN4K2qRxOBQL4N0Me5P1WLg8o8AFsDXHwT6p7iMNjxyobN+YINj zBoa7uwQ7p2gm+VBXkCoV9ctbBa1iPzQQzrmxOrutU27o9m6RkGnWUfaygDNNH+1mE7Q gmgA== X-Gm-Message-State: AOJu0YyVnoNxJeYlkW2AHaiT06HN2sv8UA+yiaUTSLf0wuW2wrU7GX/Z Cj0npUq6uj+i+4xhYeIiDY8yOQKTOn1CO6W/Hu9TWbH+AekFRa/6r/kmKw== X-Gm-Gg: ASbGnctnZVmFoFQ6TaUvHBUBp0KoOD99ubdhAZ4e8CXK3JpX2RWNqo5LZeA2+qR3wCk dfjPs9JBLTaciyjqO/UgqbnKWEkcd8IYYAgbv3i21qDsIPstq+U2SM6bOJZZ0grpCQMnesXZi1J HJEPTNMuGxFgOliQwrCttk7iP1pndk6zmK4xn5krv+bYWtvnLzpofLG4slPQ/yk39oPWjqv0Pbm NTGnLv5SEbDdFWAro017Mc4xnZ7qCFP7ApCgl2PsVmZGxZ/af8/d0J8wZfCu7b/+JyuCiNY7Ppy 0OY67p6opIs6Sk4bFlPwHGPXT8Mhxn8FdAIzLRB2FUJWrrdG06NsaF1wvqpsl/wBasNbjbmN0N7 dj2ZV/itp4ywSTX9llqUwQNtmkVc= X-Google-Smtp-Source: AGHT+IGrZSs5jrje+NpHOPn9aVGdufWDoUc2qstNrTGwp75WOtS7ijpy4wVODrvXHX8ZVXJdpPavuw== X-Received: by 2002:a05:600c:4f16:b0:439:5f04:4f8d with SMTP id 5b1f17b1804b1-43d1f235e2bmr28804135e9.12.1741961535276; Fri, 14 Mar 2025 07:12:15 -0700 (PDT) Received: from christian-Precision-5550.. (176-138-135-207.abo.bbox.fr. [176.138.135.207]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43d1fe3a8cdsm8418255e9.2.2025.03.14.07.12.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 14 Mar 2025 07:12:14 -0700 (PDT) From: Christian Couder To: git@vger.kernel.org Cc: Junio C Hamano , Jeff King , Patrick Steinhardt , Taylor Blau , Eric Sunshine , Karthik Nayak , Kristoffer Haugsbakk , "brian m . carlson" , "Randall S . Becker" , Christian Couder , Christian Couder Subject: [PATCH v5 2/3] promisor-remote: fix possible issue when no URL is advertised Date: Fri, 14 Mar 2025 15:12:02 +0100 Message-ID: <20250314141203.2548803-3-christian.couder@gmail.com> X-Mailer: git-send-email 2.49.0.rc2.36.ga8a9f9b33b In-Reply-To: <20250314141203.2548803-1-christian.couder@gmail.com> References: <20250313103859.817127-1-christian.couder@gmail.com> <20250314141203.2548803-1-christian.couder@gmail.com> Precedence: bulk X-Mailing-List: git@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In the 'KnownUrl' case, in should_accept_remote(), let's check that `remote_url` is not NULL before we use strcmp() to compare it with the local URL. This could avoid crashes if a server starts to not advertise any URL in the future. If `remote_url` is NULL, we should reject the URL. Let's also warn in this case because we warn otherwise when a remote is rejected to try to help diagnose things at the end of the function. And while we are checking that remote_url is not NULL and warning if it is, it makes sense to also help diagnose the case where remote_url is empty. Also while at it, let's spell "URL" with uppercase letters in all the warnings. Signed-off-by: Christian Couder --- promisor-remote.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/promisor-remote.c b/promisor-remote.c index ba80240f12..0b7b1ec45a 100644 --- a/promisor-remote.c +++ b/promisor-remote.c @@ -409,10 +409,15 @@ static int should_accept_remote(enum accept_promisor accept, if (accept != ACCEPT_KNOWN_URL) BUG("Unhandled 'enum accept_promisor' value '%d'", accept); + if (!remote_url || !*remote_url) { + warning(_("no or empty URL advertised for remote '%s'"), remote_name); + return 0; + } + if (!strcmp(urls->v[i], remote_url)) return 1; - warning(_("known remote named '%s' but with url '%s' instead of '%s'"), + warning(_("known remote named '%s' but with URL '%s' instead of '%s'"), remote_name, urls->v[i], remote_url); return 0; From patchwork Fri Mar 14 14:12:03 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christian Couder X-Patchwork-Id: 14016939 Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com [209.85.128.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DDAC21FFC51 for ; Fri, 14 Mar 2025 14:12:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.49 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741961540; cv=none; b=Rt+YUq+QnI5Fre/Ui4NAdgUVV6P16fbu6nFtucwB/YzVmAlMT7fkEt5QlW5wyEE/Li+P5tVrHQ/Agg5Ii/1kHXFlPkUdHEEZm3N59sbfnJzYKBrBRD19EV9HfMcnYnfUHzWWLb0qrzGuTkNXQ8v8etaetcQYB1yJqodhVcpAMIw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741961540; c=relaxed/simple; bh=W4bImaph0citfsukAEAATRJt1Kd8N7BrEMA/baDGg8Y=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ipkPl/UM0eRMN0zb6mrG0Su7W562c9h/F3ijNOa3NwjiJG9qJgbnxIowbF3wuU93R1lg/z70O9hsnT54lASjLVD6GIKYA0DhCvgWNZg04uVWfatQ8PxuVH3bT3FcpGX3Tc9CqB5UuxFjHDncOegcHdORXYCHb90/zq8FzCuDzPo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=F/G118VY; arc=none smtp.client-ip=209.85.128.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="F/G118VY" Received: by mail-wm1-f49.google.com with SMTP id 5b1f17b1804b1-43cf257158fso14713495e9.2 for ; Fri, 14 Mar 2025 07:12:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1741961537; x=1742566337; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=W9sbKceYibewCEsf+1vpBKWJ6b8kluoQ9iPfK3wbyXo=; b=F/G118VYKH8WluNHmHZNxEyTIhIb5otdsHTbB3xBW3mH9ObrbXe1TbXLtr5DKeSoKN mWe93gsERALw4DRSg455LFqlCv62v1Y5t0fujEHyw6We0ByL2UI65oCgUUA28mTI7sXX GSzePMKcXJYdi+Jz4Myywnayl6HCQZsmpUzP441wgrLCPIwVx200m99wecGrMeTEeeRE DayiSzFt8Nz1m9/0brZxm9yZeEJy65nZO7hsfdcBzQycZu5zoDv2Gt0ol8DROY5cvcbJ LbXakPGDzAIunpg5EtHqLhhQ+lJjEHos7nvNoPxC5mefoVYwjAWiHKexCf3Kc/qdoy06 zP9Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741961537; x=1742566337; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=W9sbKceYibewCEsf+1vpBKWJ6b8kluoQ9iPfK3wbyXo=; b=aLoxcr8un3orW1ilbfcMO4o/VeaX3CrER1OYky5ZShryHCiWZeeBE011IVtHM82RHH ohTuhyWF0Tn6kOcTBdbNbu4g45H7q7958rCyEZ7XmdVZkJW5Z3o8+Up6FYHYTzz1JvpY +eCA8OSXz6NTe3P5Jn2heW5Pp1QrxlGMWE/rcruSO9a9IK6HtImqqQqNe4IPeQlbkYpu ZAwAxPVndL+KRYbSbXaMmhtl0FMq1vxKAOkCMEzC3pomIU/HmxYKCzjT+Mk8cmQQGDre e2zbPUgQs59OiJIkTeD9mttBWIto3pMXEnXYQ7ah2881zxVf29EVkf2hdV7tcDJImxJz sXww== X-Gm-Message-State: AOJu0Ywenugz9acwoSyJMRNBIlwDXqrDzXyQ4AlyEK85zahlYSvSapkl BIxYhlDvDjKXbLWL402aswm4r8PFedDwPEWzdFOfo2Tf6r4TkFsj50ZNXg== X-Gm-Gg: ASbGncuojv+9lYvNLvGP8HKG7Ybjd2DUkOkECyPLLwfwklUqLjUq8xG5HI8opRs+KEw gQD8tgJe54kVysFR1VumARCi5Ie+iG/CCJLnAZhWc79lpjvb0vYHWk5ZAOFtVR2D4XQWRC47JVo HnxIv28IB83zDJR+dgKz7S9CzyJzH7IuOkZb0syCsUOMAu7kuI6O6j5W37dKITF+ZiIqiiujDrp WmZiZdhl6aNI88W1pIyuv8jvvbC/r65eKqM4uAFL+EqtMtTA/U7eoknjeRk8QVv3Z4hTqzEM/gv CAwvysXIP6w6Utjp742fwGDGASzHklnKqZFtjpU37cXeIqt0pfNTAW8vY+w3RJ2oek1chGnBjhV mAQ7+wJrFCRdAPfusst6dMNZhom4= X-Google-Smtp-Source: AGHT+IFim/iq4tFF/Zj8E/jCOs/ygIORslcrienQ5sBgRF2w7TSI7Y24SchUcm+FTyuJRXuihmxWuw== X-Received: by 2002:a05:600c:1c81:b0:43c:ea1a:720a with SMTP id 5b1f17b1804b1-43d1ec90771mr25368985e9.1.1741961536739; Fri, 14 Mar 2025 07:12:16 -0700 (PDT) Received: from christian-Precision-5550.. (176-138-135-207.abo.bbox.fr. [176.138.135.207]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43d1fe3a8cdsm8418255e9.2.2025.03.14.07.12.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 14 Mar 2025 07:12:15 -0700 (PDT) From: Christian Couder To: git@vger.kernel.org Cc: Junio C Hamano , Jeff King , Patrick Steinhardt , Taylor Blau , Eric Sunshine , Karthik Nayak , Kristoffer Haugsbakk , "brian m . carlson" , "Randall S . Becker" , Christian Couder , Christian Couder Subject: [PATCH v5 3/3] promisor-remote: compare remote names case sensitively Date: Fri, 14 Mar 2025 15:12:03 +0100 Message-ID: <20250314141203.2548803-4-christian.couder@gmail.com> X-Mailer: git-send-email 2.49.0.rc2.36.ga8a9f9b33b In-Reply-To: <20250314141203.2548803-1-christian.couder@gmail.com> References: <20250313103859.817127-1-christian.couder@gmail.com> <20250314141203.2548803-1-christian.couder@gmail.com> Precedence: bulk X-Mailing-List: git@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Because the "[remote "nick"] fetch = ..." configuration variables have the nickname in the second part, the nicknames are case sensitive, unlike the first and the third component (i.e. "remote.origin.fetch" and "Remote.origin.FETCH" are the same thing, but "remote.Origin.fetch" and "remote.origin.fetch" are different). Let's follow the way Git works in general and compare the remote names case sensitively when processing advertised remotes. Signed-off-by: Christian Couder --- Documentation/config/promisor.adoc | 4 ++-- promisor-remote.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/config/promisor.adoc b/Documentation/config/promisor.adoc index 9192acfd24..2638b01f83 100644 --- a/Documentation/config/promisor.adoc +++ b/Documentation/config/promisor.adoc @@ -26,5 +26,5 @@ promisor.acceptFromServer:: server will be accepted. By accepting a promisor remote, the client agrees that the server might omit objects that are lazily fetchable from this promisor remote from its responses - to "fetch" and "clone" requests from the client. See - linkgit:gitprotocol-v2[5]. + to "fetch" and "clone" requests from the client. Name and URL + comparisons are case sensitive. See linkgit:gitprotocol-v2[5]. diff --git a/promisor-remote.c b/promisor-remote.c index 0b7b1ec45a..5801ebfd9b 100644 --- a/promisor-remote.c +++ b/promisor-remote.c @@ -370,13 +370,13 @@ char *promisor_remote_info(struct repository *repo) /* * Find first index of 'nicks' where there is 'nick'. 'nick' is - * compared case insensitively to the strings in 'nicks'. If not found + * compared case sensitively to the strings in 'nicks'. If not found * 'nicks->nr' is returned. */ static size_t remote_nick_find(struct strvec *nicks, const char *nick) { for (size_t i = 0; i < nicks->nr; i++) - if (!strcasecmp(nicks->v[i], nick)) + if (!strcmp(nicks->v[i], nick)) return i; return nicks->nr; }