From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 10 Jul 2014 18:27:55 +0000 (-0700)
Subject: Merge branch 'dt/refs-check-refname-component-sse-fix'
X-Git-Tag: v2.1.0-rc0~50
X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/779c99fd68dcdaff7d996a1985914154a36a272c?hp=df4d7d56461c19361a6f32b633e850c7ba6e55e6

Merge branch 'dt/refs-check-refname-component-sse-fix'

Fixes to a topic that is already in 'master'.

* dt/refs-check-refname-component-sse-fix:
refs: fix valgrind suppression file
refs.c: handle REFNAME_REFSPEC_PATTERN at end of page
---

diff --git a/refs.c b/refs.c
index 20e2bf18c9..82e4842a36 100644
--- a/refs.c
+++ b/refs.c
@@ -153,6 +153,7 @@ int check_refname_format(const char *refname, int flags)
 	const __m128i tilde_lb = _mm_set1_epi8('~' - 1);
 
 	int component_count = 0;
+	int orig_flags = flags;
 
 	if (refname[0] == 0 || refname[0] == '/') {
 		/* entirely empty ref or initial ref component */
@@ -178,7 +179,7 @@ int check_refname_format(const char *refname, int flags)
 			 * End-of-page; fall back to slow method for
 			 * this entire ref.
 			 */
-			return check_refname_format_bytewise(refname, flags);
+			return check_refname_format_bytewise(refname, orig_flags);
 
 		tmp = _mm_loadu_si128((__m128i *)cp);
 		tmp1 = _mm_loadu_si128((__m128i *)(cp + 1));
diff --git a/t/valgrind/default.supp b/t/valgrind/default.supp
index 332ab1a3b3..9d51c92b74 100644
--- a/t/valgrind/default.supp
+++ b/t/valgrind/default.supp
@@ -50,10 +50,17 @@
 	fun:copy_ref
 }
 {
-	ignore-sse-check_refname_format
+	ignore-sse-check_refname_format-addr
 	Memcheck:Addr8
 	fun:check_refname_format
-	fun:cmd_check_ref_format
-	fun:handle_builtin
-	fun:main
+}
+{
+	ignore-sse-check_refname_format-cond
+	Memcheck:Cond
+	fun:check_refname_format
+}
+{
+	ignore-sse-check_refname_format-value
+	Memcheck:Value8
+	fun:check_refname_format
 }