From cce548e3e0c14b5d46c8d886c9954f4b66533ecd Mon Sep 17 00:00:00 2001 From: Russell Belfer Date: Tue, 22 Jan 2013 15:28:25 -0800 Subject: Fix case sensitivity bug with tree iterators With the new code to make tree iterators support ignore_case, there is a bug in setting the start entry for range bounded iterators where memcmp was being used instead of strncasecmp. This fixes that and expands the tree iterator test to cover the cases that were broken. --- src/iterator.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'src/iterator.c') diff --git a/src/iterator.c b/src/iterator.c index 56b262975..1c36cac78 100644 --- a/src/iterator.c +++ b/src/iterator.c @@ -255,15 +255,17 @@ static int tree_iterator__icase_map_cmp(const void *a, const void *b, void *data git_tree *tree = data; const git_tree_entry *te1 = git_tree_entry_byindex(tree, (size_t)a); const git_tree_entry *te2 = git_tree_entry_byindex(tree, (size_t)b); + return te1 ? (te2 ? git_tree_entry_icmp(te1, te2) : 1) : -1; } -static int tree_iterator__frame_start_icmp(const void *key, const void *element) +static int tree_iterator__frame_start_icmp(const void *key, const void *el) { const tree_iterator_frame *tf = (const tree_iterator_frame *)key; - const git_tree_entry *te = git_tree_entry_byindex(tf->tree, (size_t)element); + const git_tree_entry *te = git_tree_entry_byindex(tf->tree, (size_t)el); + size_t minlen = min(tf->startlen, te->filename_len); - return memcmp(tf->start, te->filename, min(tf->startlen, te->filename_len)); + return git__strncasecmp(tf->start, te->filename, minlen); } static void tree_iterator__frame_seek_start(tree_iterator_frame *tf) -- cgit v1.2.3