threads / patch / 14259

patchprevent checkout from creating branches that start with a dash

Subject: [PATCH] prevent checkout from creating branches that start with a dash

## tl;dr

4 messages between Jul 2, 2008 and Jul 2, 2008. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Bart Trojanowski· Jul 2, 2008, 15:01 UTC · lore

It was previously possible to create a -f branch with git-checkout, which could not be used or deleted.

        $ git checkout -b -f master
        Switched to a new branch "-f"

Signed-off-by: Bart Trojanowski <bart@jukie.net>
---
 branch.c           |   11 +++++++++++
 branch.h           |    5 +++++
 builtin-checkout.c |    4 ++++
 3 files changed, 20 insertions(+), 0 deletions(-)

Show changes to 3 files +20 −0

branch.c, branch.h, builtin-checkout.c

diff --git a/branch.c b/branch.c
index 56e9492..c25e362 100644
--- a/branch.c
+++ b/branch.c
@@ -170,3 +170,14 @@ void remove_branch_state(void)
 	unlink(git_path("MERGE_MSG"));
 	unlink(git_path("SQUASH_MSG"));
 }
+
+int validate_branch_name(const char *branch_name)
+{
+	if (!*branch_name)
+		return -1;
+
+	if (*branch_name == '-')
+		return -1;
+
+	return 0;
+}
diff --git a/branch.h b/branch.h
index 9f0c2a2..13999ba 100644
--- a/branch.h
+++ b/branch.h
@@ -21,4 +21,9 @@ void create_branch(const char *head, const char *name, const char *start_name,
  */
 void remove_branch_state(void);
 
+/*
+ * Check if the branch name given is well formed.  Returns 0 on success.
+ */
+int validate_branch_name(const char *branch_name);
+
 #endif
diff --git a/builtin-checkout.c b/builtin-checkout.c
index 93ea69b..f425646 100644
--- a/builtin-checkout.c
+++ b/builtin-checkout.c
@@ -553,6 +553,10 @@ int cmd_checkout(int argc, const char **argv, const char *prefix)
 	if (opts.force && opts.merge)
 		die("git checkout: -f and -m are incompatible");
 
+	if (opts.new_branch && validate_branch_name(opts.new_branch))
+		die("git checkout: '%s' does not look like a valid branch name",
+				opts.new_branch);
+
 	if (argc) {
 		const char **pathspec = get_pathspec(prefix, argv);

-- 
1.5.6.1.109.ga974cd.dirty

Brian Gernhardt· Jul 2, 2008, 15:22 UTC · re: Bart Trojanowski · lore

Re: [PATCH] prevent checkout from creating branches that start with a dash

On Jul 2, 2008, at 11:01 AM, Bart Trojanowski wrote:

Show 6 quoted lines

> +	if (opts.new_branch && validate_branch_name(opts.new_branch))
> +		die("git checkout: '%s' does not look like a valid branch name",
> +				opts.new_branch);
> +
> 	if (argc) {
> 		const char **pathspec = get_pathspec(prefix, argv);

Perhaps this should have !opts.force in that if? That way if someone needs something like "-mm" branch, they can have it.

And that way, if people really want to shoot themselves in the foot with a -f branch, they can? "git branch -f -b -f"?

~~ Brian

Johannes Sixt· Jul 2, 2008, 15:38 UTC · re: Bart Trojanowski · lore

Re: [PATCH] prevent checkout from creating branches that start with a dash

Bart Trojanowski schrieb:

Show 5 quoted lines

> It was previously possible to create a -f branch with git-checkout, which
> could not be used or deleted.
> 
>         $ git checkout -b -f master
>         Switched to a new branch "-f"

"-f" *is* a valid branch name and can be used and deleted:

$ git checkout -b -f next Switched to a new branch "-f" $ git checkout next Switched to branch "next" Your branch is ahead of the tracked remote branch 'origin/next' by 2 commits. $ git checkout -- -f Switched to branch "-f" $ git checkout next Switched to branch "next" Your branch is ahead of the tracked remote branch 'origin/next' by 2 commits. $ git branch -d -- -f Deleted branch -f.

-- Hannes

Bart Trojanowski· Jul 2, 2008, 16:34 UTC · re: Johannes Sixt · lore

Re: [PATCH] prevent checkout from creating branches that start with a dash

* Johannes Sixt <j.sixt@viscovery.net> [080702 12:00]:

Show 8 quoted lines

> Bart Trojanowski schrieb:
> > It was previously possible to create a -f branch with git-checkout, which
> > could not be used or deleted.
> > 
> >         $ git checkout -b -f master
> >         Switched to a new branch "-f"
> 
> "-f" *is* a valid branch name and can be used and deleted:

Thanks, that I didn't know.

-Bart

-- 
				WebSig: http://www.jukie.net/~bart/sig/