git-checkout: disallow --detach on unborn branch
authorChris Webb <chris@arachsys.com>
Tue, 26 Jun 2012 15:06:42 +0000 (16:06 +0100)
committerJunio C Hamano <gitster@pobox.com>
Tue, 26 Jun 2012 18:11:14 +0000 (11:11 -0700)
commit8ced1aa08f9e1798b2b3fec41a0598ef79b477fe
tree4cad32f246e34e8686b0dae6045c60875de8f4ae
parentabe199808c6586047fb7255b80e3d17ffc26bf6c
git-checkout: disallow --detach on unborn branch

abe199808c (git checkout -b: allow switching out of an unborn branch)
introduced a bug demonstrated by

  git checkout --orphan foo
  git checkout --detach
  git symbolic-ref HEAD

which gives 'refs/heads/(null)'.

This happens because we strbuf_addf(&branch_ref, "refs/heads/%s",
opts->new_branch) when opts->new_branch can be NULL for --detach.

Catch and forbid this case, adding a test to t2017 to catch it in
future.

Signed-off-by: Chris Webb <chris@arachsys.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/checkout.c
t/t2017-checkout-orphan.sh