git_exec_path, execv_git_cmd: ignore empty environment variables
authorDmitry V. Levin <ldv@altlinux.org>
Mon, 29 May 2006 00:34:34 +0000 (04:34 +0400)
committerJunio C Hamano <junkio@cox.net>
Wed, 31 May 2006 04:49:01 +0000 (21:49 -0700)
Ignoring empty environment variables is good common practice.
Ignoring --exec-path with empty argument won't harm, too:
if user means current directory, there is a "--exec-path=."

Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
exec_cmd.c

index 12eb36494ca467169aff4db35335b3fdadd4cbee..c1539d12ce9e350811b5b110206f5577a151e8ef 100644 (file)
@@ -21,7 +21,7 @@ const char *git_exec_path(void)
                return current_exec_path;
 
        env = getenv("GIT_EXEC_PATH");
-       if (env) {
+       if (env && *env) {
                return env;
        }
 
@@ -43,7 +43,7 @@ int execv_git_cmd(const char **argv)
                const char *exec_dir = paths[i];
                const char *tmp;
 
-               if (!exec_dir) continue;
+               if (!exec_dir || !*exec_dir) continue;
 
                if (*exec_dir != '/') {
                        if (!getcwd(git_command, sizeof(git_command))) {