git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v2 0/3] fetch: fix segfault, missing docs in --negotiate-only

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Jul 8, 2021, 10:53 UTC
Message-ID
<cover-0.3-00000000000-20210708T105034Z-avarab@gmail.com>
In-Reply-To
<cover-0.3-00000000000-20210630T163329Z-avarab@gmail.com>

Fix a segfault in recent code, per v1's: https://lore.kernel.org/git/cover-0.3-00000000000-20210630T163329Z-avarab@gmail.com/

Fixes a stray "cat err" that snuck out of the lab and into the test, spotted by Jonathan Tan.

Junio: You picked this up as ab/fetch-negotiate-segv-fix and it's
marked as "Will merge to 'next'", but it's not there yet. Hopefully
you'll see this before it's merged down. Sorry about the delay.
Ævar Arnfjörð Bjarmason (3):
  send-pack.c: move "no refs in common" abort earlier
  fetch: document the --negotiate-only option
  fetch: fix segfault in --negotiate-only without --negotiation-tip=*
 Documentation/config/fetch.txt  |  3 ++-
 Documentation/fetch-options.txt | 13 +++++++++++--
 builtin/fetch.c                 |  3 +++
 send-pack.c                     | 11 ++++++-----
 t/t5702-protocol-v2.sh          | 16 ++++++++++++++++
 5 files changed, 38 insertions(+), 8 deletions(-)
Range-diff against v1:
1:  1cd8b98d3d6 = 1:  10375a6484e send-pack.c: move "no refs in common" abort earlier
2:  7ff734ed70a = 2:  f86f4fc0e6f fetch: document the --negotiate-only option
3:  38930024d95 ! 3:  491d72c35a0 fetch: fix segfault in --negotiate-only without --negotiation-tip=*
    @@ t/t5702-protocol-v2.sh: setup_negotiate_only () {
     +	test_must_fail git -c protocol.version=2 -C client fetch \
     +		--negotiate-only \
     +		origin 2>err.actual &&
    -+	cat err &&
     +	test_cmp err.expect err.actual
     +'
     +
-- 
2.32.0.636.g43e71d69cff
Previous: Ævar Arnfjörð BjarmasonNext: Ævar Arnfjörð Bjarmason
Message 6 of 9 in “fetch: fix segfault, missing docs in --negotiate-only”
  1. 0/3 fetch: fix segfault, missing docs in --negotiate-onlyÆvar Arnfjörð Bjarmason, Jun 30, 2021
  2. 1/3 send-pack.c: move "no refs in common" abort earlierÆvar Arnfjörð Bjarmason, Jun 30, 2021
  3. 3/3 fetch: fix segfault in --negotiate-only without --negotiation-tip=*Ævar Arnfjörð Bjarmason, Jun 30, 2021
  4. Jonathan TanJun 30, 2021
  5. 2/3 fetch: document the --negotiate-only optionÆvar Arnfjörð Bjarmason, Jun 30, 2021
  6. 0/3 fetch: fix segfault, missing docs in --negotiate-onlyÆvar Arnfjörð Bjarmason, Jul 8, 2021
  7. 1/3 send-pack.c: move "no refs in common" abort earlierÆvar Arnfjörð Bjarmason, Jul 8, 2021
  8. 2/3 fetch: document the --negotiate-only optionÆvar Arnfjörð Bjarmason, Jul 8, 2021
  9. 3/3 fetch: fix segfault in --negotiate-only without --negotiation-tip=*Ævar Arnfjörð Bjarmason, Jul 8, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.