Skip to content

Commit

Permalink
Merge branch 'dt/refs-check-refname-component-sse-fix'
Browse files Browse the repository at this point in the history
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
  • Loading branch information
Junio C Hamano committed Jul 10, 2014
2 parents df4d7d5 + a8d9fea commit 779c99f
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
3 changes: 2 additions & 1 deletion refs.c
Original file line number Diff line number Diff line change
Expand Up @@ -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 */
Expand All @@ -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));
Expand Down
15 changes: 11 additions & 4 deletions t/valgrind/default.supp
Original file line number Diff line number Diff line change
Expand Up @@ -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
}

0 comments on commit 779c99f

Please sign in to comment.