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

[PATCH] clone: don't use an integer as a NULL pointer

From
Ramsay Jones <ramsay@ramsayjones.plus.com>
Date
May 25, 2016, 23:12 UTC
Message-ID
<5746316E.8020902@ramsayjones.plus.com>
Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
---
Hi Stefan,

If you need to re-roll your 'sb/submodule-default-paths' branch, could you please squash this into the relevant patch. (commit 8efbe28b, "clone: add --init-submodule=<pathspec> switch", 23-05-2016).

Thanks!

ATB, Ramsay Jones

 builtin/clone.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/clone.c b/builtin/clone.c
index 22b6eac..a056f72 100644
--- a/builtin/clone.c
+++ b/builtin/clone.c
@@ -925,7 +925,7 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
 		struct strbuf sb = STRBUF_INIT;
 		for_each_string_list_item(item, &init_submodules) {
 			strbuf_addf(&sb, "submodule.defaultUpdatePath=%s", item->string);
-			string_list_append(&option_config, strbuf_detach(&sb, 0));
+			string_list_append(&option_config, strbuf_detach(&sb, NULL));
 		}
 	}
 
-- 
2.8.0
Next: Stefan Beller
Message 1 of 3 in “clone: don't use an integer as a NULL pointer”
  1. clone: don't use an integer as a NULL pointerRamsay Jones, May 25, 2016
  2. Stefan BellerMay 25, 2016
  3. Ramsay JonesMay 26, 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.