summaryrefslogtreecommitdiff
path: root/connect.c
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2017-07-28 19:25:45 (GMT)
committerJunio C Hamano <gitster@pobox.com>2017-07-28 22:51:56 (GMT)
commit2491f77b90c2e5d47acbe7472c17e7de0af74f63 (patch)
treeba20f55e5f188c839bb952f7b1a8428c370f9374 /connect.c
parent2d90add5ad216807ec1433e5367fae730e74a4cb (diff)
downloadgit-2491f77b90c2e5d47acbe7472c17e7de0af74f63.zip
git-2491f77b90c2e5d47acbe7472c17e7de0af74f63.tar.gz
git-2491f77b90c2e5d47acbe7472c17e7de0af74f63.tar.bz2
connect: factor out "looks like command line option" check
We reject hostnames that start with a dash because they may be confused for command-line options. Let's factor out that notion into a helper function, as we'll use it in more places. And while it's simple now, it's not clear if some systems might need more complex logic to handle all cases. Signed-off-by: Jeff King <peff@peff.net> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'connect.c')
-rw-r--r--connect.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/connect.c b/connect.c
index 0e8e05d..a0091ac 100644
--- a/connect.c
+++ b/connect.c
@@ -754,7 +754,7 @@ struct child_process *git_connect(int fd[2], const char *url,
return NULL;
}
- if (ssh_host[0] == '-')
+ if (looks_like_command_line_option(ssh_host))
die("strange hostname '%s' blocked", ssh_host);
ssh = getenv("GIT_SSH_COMMAND");