Signed-off-by: Sun He <>

Check the limit.h of linux and find out that the MACRO
#define PATH_MAX        4096    /* # chars in a path name including nul */
So if the magic number 40 is just the size it should be. (e.g. hash code)
It may bring bugs with the length(4056) of long name(gitdirenv).
As gitdirenv could be set by GIT_DIR_ENVIRONMENT.
If it is a bug, it will almost never occur.
But I need your help to know if there is the PATH_MAX of git is the mirror of 
PATH_MAX of linux and if this fix is right?
If it was, there may be many places like "PATH_MAX + 1" could be replaced by
just "PATH_MAX". And there may be many places like this.

He Sun

 setup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/setup.c b/setup.c
index cffb6d6..1511612 100644
--- a/setup.c
+++ b/setup.c
@@ -395,7 +395,7 @@ static const char *setup_explicit_git_dir(const char 
        char *gitfile;
        int offset;
-       if (PATH_MAX - 40 < strlen(gitdirenv))
+       if (PATH_MAX - 41 < strlen(gitdirenv))
                die("'$%s' too big", GIT_DIR_ENVIRONMENT);
        gitfile = (char*)read_gitfile(gitdirenv);

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to