[PATCH] Fix off-by-one in read_tree_recursive

Found by valgrind.

Signed-off-by: Johannes Schindelin <redacted>
---
 tree.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/tree.c b/tree.c
index 0d703a0..5ab90af 100644
--- a/tree.c
+++ b/tree.c

@@ -62,6 +62,7 @@ static int match_tree_entry(const char *base, int baselen, const char *path, uns
 				continue;
 			/* pathspecs match only at the directory boundaries */
 			if (!matchlen ||
+			    baselen == matchlen ||
 			    base[matchlen] == '/' ||
 			    match[matchlen - 1] == '/')
 				return 1;

-- 
1.6.2.1.493.g67cf3

On Apr 20, 2009, at 12:44 PM, Johannes Schindelin wrote:

Found by valgrind.

How are you running valgrind?  Do you have a test suite for it?  A  
make target?

xoa


--
Andy Lester => andy@petdance.com => www.theworkinggeek.com => AIM:petdance

Hi,

On Mon, 20 Apr 2009, Andy Lester wrote:

On Apr 20, 2009, at 12:44 PM, Johannes Schindelin wrote:

quoted

Found by valgrind.

How are you running valgrind?  Do you have a test suite for it?  A make 
target?

Yes, it was merged into Git:

http://repo.or.cz/w/alt-git.git?a=commitdiff;h=8a61097cde387870921e822bdf90b4ffd3792845

Hth,
Dscho

On Apr 20, 2009, at 1:34 PM, Johannes Schindelin wrote:

Yes, it was merged into Git:

http://repo.or.cz/w/alt-git.git?a=commitdiff;h=8a61097cde387870921e822bdf90b4ffd3792845

Very cool.  In my Copious Free Time, I'm trying to get a splint target  
built up.

xoxo,
Andy

--
Andy Lester => andy@petdance.com => www.theworkinggeek.com => AIM:petdance