summaryrefslogtreecommitdiff
path: root/refs.c
diff options
context:
space:
mode:
authorMartin Ågren <martin.agren@gmail.com>2018-05-10 19:29:56 (GMT)
committerJunio C Hamano <gitster@pobox.com>2018-05-13 01:36:11 (GMT)
commitdb0210d445963e5d85f98e48d6a93b971779d449 (patch)
treeaa73521bfdf5c6203a686238ef84892ab8b9549e /refs.c
parent65eb8fc344205a4039b989f07367f345101bbf28 (diff)
downloadgit-db0210d445963e5d85f98e48d6a93b971779d449.zip
git-db0210d445963e5d85f98e48d6a93b971779d449.tar.gz
git-db0210d445963e5d85f98e48d6a93b971779d449.tar.bz2
refs: handle zero oid for pseudorefs
According to the documentation, it is possible to "specify 40 '0' or an empty string as <oldvalue> to make sure that the ref you are creating does not exist." But in the code for pseudorefs, we do not implement this, as demonstrated by the failing tests added in the previous commit. If we fail to read the old ref, we immediately die. But a failure to read would actually be a good thing if we have been given the zero oid. With the zero oid, allow -- and even require -- the ref-reading to fail. This implements the "make sure that the ref ... does not exist" part of the documentation and fixes both failing tests from the previous commit. Since we have a `strbuf err` for collecting errors, let's use it and signal an error to the caller instead of dying hard. Reported-by: Rafael Ascensão <rafa.almas@gmail.com> Helped-by: Rafael Ascensão <rafa.almas@gmail.com> Signed-off-by: Martin Ågren <martin.agren@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'refs.c')
-rw-r--r--refs.c16
1 files changed, 13 insertions, 3 deletions
diff --git a/refs.c b/refs.c
index 7820a52..26af07f 100644
--- a/refs.c
+++ b/refs.c
@@ -681,9 +681,19 @@ static int write_pseudoref(const char *pseudoref, const struct object_id *oid,
if (old_oid) {
struct object_id actual_old_oid;
- if (read_ref(pseudoref, &actual_old_oid))
- die("could not read ref '%s'", pseudoref);
- if (oidcmp(&actual_old_oid, old_oid)) {
+ if (read_ref(pseudoref, &actual_old_oid)) {
+ if (!is_null_oid(old_oid)) {
+ strbuf_addf(err, "could not read ref '%s'",
+ pseudoref);
+ rollback_lock_file(&lock);
+ goto done;
+ }
+ } else if (is_null_oid(old_oid)) {
+ strbuf_addf(err, "ref '%s' already exists",
+ pseudoref);
+ rollback_lock_file(&lock);
+ goto done;
+ } else if (oidcmp(&actual_old_oid, old_oid)) {
strbuf_addf(err, "unexpected object ID when writing '%s'",
pseudoref);
rollback_lock_file(&lock);