[PATCH] Use correct value when hinting strbuf_read()
From: Fredrik Gustafsson <hidden>
Date: 2016-06-15 22:51:29
Subsystem:
the rest · Maintainer:
Linus Torvalds
From: Fredrik Gustafsson <hidden>
Date: 2016-06-15 22:51:29
Subsystem:
the rest · Maintainer:
Linus Torvalds
The git strbuf allows for each read to hint about the size of the string. In this case the the string can never be longer than 41 characters, as it cannot contain more than a single hex-sha1 and a newline. So let's use 41 instead of 1024 to reduce the memory footprint. Signed-off-by: Fredrik Gustafsson <redacted> Mentored-by: Jens Lehmann [off-list ref] Mentored-by: Heiko Voigt [off-list ref] --- submodule.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/submodule.c b/submodule.c
index b6dec70..86baf42 100644
--- a/submodule.c
+++ b/submodule.c@@ -326,7 +326,7 @@ static int is_submodule_commit_present(const char *path, unsigned char sha1[20]) cp.no_stdin = 1; cp.out = -1; cp.dir = path; - if (!run_command(&cp) && !strbuf_read(&buf, cp.out, 1024)) + if (!run_command(&cp) && !strbuf_read(&buf, cp.out, 41)) is_present = 1; close(cp.out);
--
1.7.5.1.229.g455f