git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: GIT_ASKPASS absolute path detection bug on Windows

From
brian m. carlson <sandals@crustytoothpaste.net>
Date
Mar 22, 2020, 16:59 UTC
Message-ID
<20200322165915.GA6499@camp.crustytoothpaste.net>
In-Reply-To
<CANPdQvK4LRZ2b_K0QuWErxNcc2wpW0W4zvmj3HzKDnOvgBNHfw@mail.gmail.com>
On 2020-03-22 at 11:44:33, András Kucsma wrote:
Show 26 quoted lines
> My proposal patch is to take advantage of find_last_dir_sep function's
> OS specific directory separator knowledge.
> I posted the diff below, which is also available on github here:
> https://github.com/git/git/compare/maint...r0mai:fix-prepare_cmd-windows-maint
> 
> diff --git a/run-command.c b/run-command.c
> index f5e1149f9b..9fcc12ebf9 100644
> --- a/run-command.c
> +++ b/run-command.c
> @@ -421,12 +421,12 @@ static int prepare_cmd(struct argv_array *out,
> const struct child_process *cmd)
>      }
> 
>      /*
> -     * If there are no '/' characters in the command then perform a path
> -     * lookup and use the resolved path as the command to exec.  If there
> -     * are '/' characters, we have exec attempt to invoke the command
> -     * directly.
> +     * If there are no dir separator characters in the command then perform
> +     * a path lookup and use the resolved path as the command to exec. If
> +     * there are dir separator characters, we have exec attempt to invoke
> +     * the command directly.
>       */
> -    if (!strchr(out->argv[1], '/')) {
> +    if (find_last_dir_sep(out->argv[1]) == NULL) {
>          char *program = locate_in_PATH(out->argv[1]);

This function (locate_in_PATH) specifically says it is not to be used on Windows because it doesn't work properly there due to file extensions. I'm pretty sure a proper solution would involve touching that as well, although your solution does indeed fix the issue you reported. That function also uses a colon-separated PATH, which I'm not sure will work in all cases on Windows (although maybe it will).

From looking at this earlier, I think the problem here is that we're trying to use the Unix codepaths (on Cygwin) and then expecting those to handle Windows-style paths, which they aren't intended to do. This is likely one of many problems on Cygwin.

-- 
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
Previous: András KucsmaNext: Torsten Bögershausen
Message 4 of 9 in “GIT_ASKPASS absolute path detection bug on Windows”
  1. András KucsmaMar 21, 2020
  2. Torsten BögershausenMar 22, 2020
  3. András KucsmaMar 22, 2020
  4. brian m. carlsonMar 22, 2020
  5. Torsten BögershausenMar 22, 2020
  6. András KucsmaMar 22, 2020
  7. Achim GratzMar 22, 2020
  8. Torsten BögershausenMar 23, 2020
  9. András KucsmaMar 23, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.