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

[PATCH v3 08/25] upload-pack: tighten number parsing at "deepen" lines

From
Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Date
Feb 23, 2016, 13:44 UTC
Message-ID
<1456235103-26317-9-git-send-email-pclouds@gmail.com>
In-Reply-To
<1456235103-26317-1-git-send-email-pclouds@gmail.com>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
---
 upload-pack.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/upload-pack.c b/upload-pack.c
index 257ad48..9f14933 100644
--- a/upload-pack.c
+++ b/upload-pack.c
@@ -641,9 +641,9 @@ static void receive_needs(void)
 			continue;
 		}
 		if (skip_prefix(line, "deepen ", &arg)) {
-			char *end;
+			char *end = NULL;
 			depth = strtol(arg, &end, 0);
-			if (end == arg || depth <= 0)
+			if (!end || *end || depth <= 0)
 				die("Invalid deepen: %s", line);
 			continue;
 		}
-- 
2.7.1.532.gd9e3aaa
Previous: Nguyễn Thái Ngọc DuyNext: Nguyễn Thái Ngọc Duy
Message 9 of 26 in “More flexibility in making shallow clones”
  1. 00/25 More flexibility in making shallow clonesNguyễn Thái Ngọc Duy, Feb 23, 2016
  2. 01/25 remote-curl.c: convert fetch_git() to use argv_arrayNguyễn Thái Ngọc Duy, Feb 23, 2016
  3. 02/25 transport-helper.c: refactor set_helper_option()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  4. 03/25 upload-pack: move shallow deepen code out of receive_needs()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  5. 04/25 upload-pack: move "shallow" sending code out of deepen()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  6. 05/25 upload-pack: remove unused variable "backup"Nguyễn Thái Ngọc Duy, Feb 23, 2016
  7. 06/25 upload-pack: move "unshallow" sending code out of deepen()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  8. 07/25 upload-pack: use skip_prefix() instead of starts_with()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  9. 08/25 upload-pack: tighten number parsing at "deepen" linesNguyễn Thái Ngọc Duy, Feb 23, 2016
  10. 09/25 upload-pack: move rev-list code out of check_non_tip()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  11. 10/25 fetch-pack: use skip_prefix() instead of starts_with()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  12. 11/25 fetch-pack: use a common function for verbose printingNguyễn Thái Ngọc Duy, Feb 23, 2016
  13. 12/25 fetch-pack: use a separate flag for fetch in deepening modeNguyễn Thái Ngọc Duy, Feb 23, 2016
  14. 13/25 shallow.c: implement a generic shallow boundary finder based on rev-listNguyễn Thái Ngọc Duy, Feb 23, 2016
  15. 14/25 upload-pack: add deepen-since to cut shallow repos based on timeNguyễn Thái Ngọc Duy, Feb 23, 2016
  16. 15/25 fetch: define shallow boundary with --shallow-sinceNguyễn Thái Ngọc Duy, Feb 23, 2016
  17. 16/25 clone: define shallow clone boundary based on time with --shallow-sinceNguyễn Thái Ngọc Duy, Feb 23, 2016
  18. 17/25 t5500, t5539: tests for shallow depth since a specific dateNguyễn Thái Ngọc Duy, Feb 23, 2016
  19. 18/25 refs: add expand_ref()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  20. 19/25 upload-pack: support define shallow boundary by excluding revisionsNguyễn Thái Ngọc Duy, Feb 23, 2016
  21. 20/25 fetch: define shallow boundary with --shallow-excludeNguyễn Thái Ngọc Duy, Feb 23, 2016
  22. 21/25 clone: define shallow clone boundary with --shallow-excludeNguyễn Thái Ngọc Duy, Feb 23, 2016
  23. 22/25 t5500, t5539: tests for shallow depth excluding a refNguyễn Thái Ngọc Duy, Feb 23, 2016
  24. 23/25 upload-pack: split check_unreachable() in two, prep for get_reachable_list()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  25. 24/25 upload-pack: add get_reachable_list()Nguyễn Thái Ngọc Duy, Feb 23, 2016
  26. 25/25 fetch, upload-pack: --deepen=N extends shallow boundary by N commitsNguyễn Thái Ngọc Duy, Feb 23, 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.