git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 2/3] rename_ref(): handle the case when the reflog of a ref does not exist

From
Miklos Vajna <vmiklos@frugalware.org>
Date
Oct 26, 2008, 02:33 UTC
Message-ID
<b06bf89aeeb844903e16f7c231c6add011a910bf.1224987945.git.vmiklos@frugalware.org>
In-Reply-To
<39b6bd01cf753d48a803e1d5908e543b66899c77.1224987944.git.vmiklos@frugalware.org>

We tried to check if a reflog of a ref is a symlink without first checking if it exists, which is a bug.

Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
---
 refs.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/refs.c b/refs.c
index 70c0967..3c39a31 100644
--- a/refs.c
+++ b/refs.c
@@ -975,7 +975,7 @@ int rename_ref(const char *oldref, const char *newref, const char *logmsg)
 	const char *symref = NULL;
 	int is_symref = 0;
 
-	if (S_ISLNK(loginfo.st_mode))
+	if (log && S_ISLNK(loginfo.st_mode))
 		return error("reflog for %s is a symlink", oldref);
 
 	symref = resolve_ref(oldref, orig_sha1, 1, &flag);
-- 
1.6.0.2
Previous: Miklos VajnaNext: Miklos Vajna
Message 3 of 6 in “symref rename/delete fixes”
  1. 0/3 symref rename/delete fixesMiklos Vajna, Oct 26, 2008
  2. 1/3 Fix git branch -m for symrefs.Miklos Vajna, Oct 26, 2008
  3. 2/3 rename_ref(): handle the case when the reflog of a ref does not existMiklos Vajna, Oct 26, 2008
  4. 3/3 Fix git update-ref --no-deref -d.Miklos Vajna, Oct 26, 2008
  5. Junio C HamanoOct 27, 2008
  6. Miklos VajnaOct 27, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.