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

[PATCH v2 1/2] cherry-pick: demonstrate a segmentation fault

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Nov 26, 2016, 12:48 UTC
Message-ID
<f1a1fbf148e1f0106f47ea0153c4eb0dfa3bd21b.1480164459.git.johannes.schindelin@gmx.de>
In-Reply-To
<cover.1480164459.git.johannes.schindelin@gmx.de>

In https://github.com/git-for-windows/git/issues/952, a complicated scenario was described that leads to a segmentation fault in cherry-pick.

It boils down to a certain code path involving a renamed file that is dirty, for which `refresh_cache_entry()` returns `NULL`, and that `NULL` not being handled properly.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
 t/t3501-revert-cherry-pick.sh | 12 ++++++++++++
 1 file changed, 12 insertions(+)
diff --git a/t/t3501-revert-cherry-pick.sh b/t/t3501-revert-cherry-pick.sh
index 394f0005a1..d7b4251234 100755
--- a/t/t3501-revert-cherry-pick.sh
+++ b/t/t3501-revert-cherry-pick.sh
@@ -141,4 +141,16 @@ test_expect_success 'cherry-pick "-" works with arguments' '
 	test_cmp expect actual
 '
 
+test_expect_failure 'cherry-pick works with dirty renamed file' '
+	test_commit to-rename &&
+	git checkout -b unrelated &&
+	test_commit unrelated &&
+	git checkout @{-1} &&
+	git mv to-rename.t renamed &&
+	test_tick &&
+	git commit -m renamed &&
+	echo modified >renamed &&
+	git cherry-pick refs/heads/unrelated
+'
+
 test_done
-- 
2.11.0.rc3.windows.1
Previous: Johannes SchindelinNext: Johannes Schindelin
Message 7 of 11 in “Fix segmentation fault with cherry-pick”
  1. 0/2 Fix segmentation fault with cherry-pickJohannes Schindelin, Nov 25, 2016
  2. 1/2 cherry-pick: demonstrate a segmentation faultJohannes Schindelin, Nov 25, 2016
  3. Johannes SchindelinNov 26, 2016
  4. 2/2 Avoid a segmentation fault with renaming mergesJohannes Schindelin, Nov 25, 2016
  5. Johannes SchindelinNov 25, 2016
  6. 0/2 Fix segmentation fault with cherry-pickJohannes Schindelin, Nov 26, 2016
  7. 1/2 cherry-pick: demonstrate a segmentation faultJohannes Schindelin, Nov 26, 2016
  8. 2/2 Avoid a segmentation fault with renaming mergesJohannes Schindelin, Nov 26, 2016
  9. Johannes SchindelinNov 26, 2016
  10. Junio C HamanoNov 28, 2016
  11. Junio C HamanoNov 28, 2016

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.