Andrew's git
/
gitweb.git
/ diff
summary
|
log
|
commit
| diff |
tree
commit
grep
author
committer
pickaxe
?
re
Merge branch 'mr/packed-ref-store-fix' into maint
author
Junio C Hamano
<gitster@pobox.com>
Thu, 22 Mar 2018 21:24:10 +0000
(14:24 -0700)
committer
Junio C Hamano
<gitster@pobox.com>
Thu, 22 Mar 2018 21:24:10 +0000
(14:24 -0700)
Crash fix for a corner case where an error codepath tried to unlock
what it did not acquire lock on.
* mr/packed-ref-store-fix:
files_initial_transaction_commit(): only unlock if locked
refs/files-backend.c
patch
|
blob
|
history
raw
(from parent 1:
721dce0
)
diff --git
a/refs/files-backend.c
b/refs/files-backend.c
index f75d960e1982a2db5dc8828c6f1b10ffe64fda48..bec8e30e9e3e2995739dfff7cc971f8de623610d 100644
(file)
--- a/
refs/files-backend.c
+++ b/
refs/files-backend.c
@@
-2931,13
+2931,12
@@
static int files_initial_transaction_commit(struct ref_store *ref_store,
if (initial_ref_transaction_commit(packed_transaction, err)) {
ret = TRANSACTION_GENERIC_ERROR;
- goto cleanup;
}
+ packed_refs_unlock(refs->packed_ref_store);
cleanup:
if (packed_transaction)
ref_transaction_free(packed_transaction);
- packed_refs_unlock(refs->packed_ref_store);
transaction->state = REF_TRANSACTION_CLOSED;
string_list_clear(&affected_refnames, 0);
return ret;