Skip to content

Commit

Permalink
Merge branch 'rs/lock-correct-ref-during-delete' into maint
Browse files Browse the repository at this point in the history
* rs/lock-correct-ref-during-delete:
  refs: lock symref that is to be deleted, not its target
  • Loading branch information
Junio C Hamano committed Nov 20, 2012
2 parents e87719f + 547d058 commit c5afc5b
Showing 1 changed file with 5 additions and 13 deletions.
18 changes: 5 additions & 13 deletions refs.c
Original file line number Diff line number Diff line change
Expand Up @@ -1753,26 +1753,18 @@ int delete_ref(const char *refname, const unsigned char *sha1, int delopt)
struct ref_lock *lock;
int err, i = 0, ret = 0, flag = 0;

lock = lock_ref_sha1_basic(refname, sha1, 0, &flag);
lock = lock_ref_sha1_basic(refname, sha1, delopt, &flag);
if (!lock)
return 1;
if (!(flag & REF_ISPACKED) || flag & REF_ISSYMREF) {
/* loose */
const char *path;

if (!(delopt & REF_NODEREF)) {
i = strlen(lock->lk->filename) - 5; /* .lock */
lock->lk->filename[i] = 0;
path = lock->lk->filename;
} else {
path = git_path("%s", refname);
}
err = unlink_or_warn(path);
i = strlen(lock->lk->filename) - 5; /* .lock */
lock->lk->filename[i] = 0;
err = unlink_or_warn(lock->lk->filename);
if (err && errno != ENOENT)
ret = 1;

if (!(delopt & REF_NODEREF))
lock->lk->filename[i] = '.';
lock->lk->filename[i] = '.';
}
/* removing the loose one could have resurrected an earlier
* packed one. Also, if it was not loose we need to repack
Expand Down

0 comments on commit c5afc5b

Please sign in to comment.