builtin-for-each-ref.c::copy_name() - do not overstep the buffer.

This was introduced during xmemdupz() conversion. Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-mirror · Sep 19, 2007 · 6b30852 · 6b30852
1 parent ca03283
commit 6b30852
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/builtin-for-each-ref.c b/builtin-for-each-ref.c
@@ -309,7 +309,7 @@ static const char *copy_line(const char *buf)
 static const char *copy_name(const char *buf)
 {
 	const char *cp;
-	for (cp = buf; *cp != '\n'; cp++) {
+	for (cp = buf; *cp && *cp != '\n'; cp++) {
 		if (!strncmp(cp, " <", 2))
 			return xmemdupz(buf, cp - buf);
 	}