]> git.ipfire.org Git - thirdparty/git.git/commitdiff
normalize_path_copy(): document "dst" size expectations
authorJeff King <peff@peff.net>
Thu, 30 Jan 2020 09:52:19 +0000 (04:52 -0500)
committerJunio C Hamano <gitster@pobox.com>
Thu, 30 Jan 2020 21:45:58 +0000 (13:45 -0800)
We take a "dst" buffer to write into, but there's no matching "len"
parameter. The hidden assumption is that normalizing always makes things
smaller, so we're OK as long as "dst" is at least as big as "src". Let's
document that explicitly.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
path.c

diff --git a/path.c b/path.c
index a76eec8b968d81904928f83aca789757117cac0f..88cf59300738a2f10750ff458fa35b1211f00ab8 100644 (file)
--- a/path.c
+++ b/path.c
@@ -1077,6 +1077,8 @@ const char *remove_leading_path(const char *in, const char *prefix)
 
 /*
  * It is okay if dst == src, but they should not overlap otherwise.
+ * The "dst" buffer must be at least as long as "src"; normalizing may shrink
+ * the size of the path, but will never grow it.
  *
  * Performs the following normalizations on src, storing the result in dst:
  * - Ensures that components are separated by '/' (Windows only)