Karthik Nayak <karthik.188@xxxxxxxxx> writes: >> diff --git a/refs/files-backend.c b/refs/files-backend.c >> index a098d14ea00..97473f377d1 100644 >> --- a/refs/files-backend.c >> +++ b/refs/files-backend.c >> @@ -794,8 +794,10 @@ static int lock_raw_ref(struct files_ref_store *refs, >> */ >> if (refs_verify_refname_available( >> refs->packed_ref_store, refname, >> - extras, NULL, err)) >> + extras, NULL, err)) { >> + ret = TRANSACTION_NAME_CONFLICT; >> goto error_return; >> + } >> } >> >> ret = 0; >> > > Shouldn't we also do this change in `lock_ref_oid_basic` where we gather > the same lock again for creating the reflog entry? An interesting question.