From 9734b74a8f9e327e02762024596961209cf989d5 Mon Sep 17 00:00:00 2001 From: Jeff King Date: Thu, 30 Jan 2020 04:52:19 -0500 Subject: [PATCH] normalize_path_copy(): document "dst" size expectations 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 Signed-off-by: Junio C Hamano --- path.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/path.c b/path.c index a76eec8b96..88cf593007 100644 --- 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)