clone: validate --origin option before use

Providing a bad origin name to `git clone` currently reports an 'invalid refspec' error instead of a more explicit message explaining that the `--origin` option was malformed. This behavior dates back to since 8434c2f1 (Build in clone, 2008-04-27). Reintroduce validation for the provided `--origin` option, but notably _don't_ include a multi-level check (e.g. "foo/bar") that was present in the original `git-clone.sh`. `git remote` allows multi-level remote names since at least 46220ca100 (remote.c: Fix overtight refspec validation, 2008-03-20), so that appears to be the desired behavior. Helped-by: Junio C Hamano <gitster@pobox.com> Helped-by: Derrick Stolee <stolee@gmail.com> Helped-by: Jeff King <peff@peff.net> Signed-off-by: Sean Barag <sean@barag.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Sean Barag committed Oct 1, 2020 at 03:46 UTC ebe7e28a3600f4e67e9a1781e335adb36f3f139b

1 file changed +3

builtin/clone.c

@@ -1012,6 +1012,9 @@ int cmd_clone(int argc, const char **argv, const char *prefix)

1012 if (!option_origin)

1013 option_origin = "origin";

1014

1015 + if (!valid_remote_name(option_origin))

1016 + die(_("'%s' is not a valid remote name"), option_origin);

1017 +

1018 repo_name = argv[0];

1019

1020 path = get_repo_path(repo_name, &is_bundle);