commit 16c2bfbb449a90db00a46984d7dd7f735caa1d56
author Lars Hjemli <hjemli@gmail.com> Wed Nov 29 21:44:56 2006 +0100
committer Junio C Hamano <junkio@cox.net> Tue Dec 05 23:50:57 2006 -0800
tree 6d268e2843256244951d6e07ba7473f3156aa201
parent c976d415e5352886f0650f8e2edba81866c38587

rename_ref: use lstat(2) when testing for symlink

The current check for symlinked reflogs was based on stat(2), which is
utterly embarrassing.

Fix it, and add a matching testcase.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>

refs.c

diff --git a/refs.c b/refs.c
index cdedb45..c23561e 100644
--- a/refs.c
+++ b/refs.c
@@ -792,7 +792,7 @@
 	struct ref_lock *lock;
 	char msg[PATH_MAX*2 + 100];
 	struct stat loginfo;
-	int log = !stat(git_path("logs/%s", oldref), &loginfo);
+	int log = !lstat(git_path("logs/%s", oldref), &loginfo);
 
 	if (S_ISLNK(loginfo.st_mode))
 		return error("reflog for %s is a symlink", oldref);