From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Wed, 23 Jul 2008 22:19:19 +0000 (+0000)
Subject: windows-btree-enum-20080723
X-Git-Tag: openafs-devel-1_5_61~943
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=a9c00f2638d44e1d077fbd953bf153df9cc319c4;p=packages%2Fo%2Fopenafs.git

windows-btree-enum-20080723

LICENSE MIT

Fix next enum to return an invalid param error when beyond the end
of the list.
---

diff --git a/src/WINNT/afsd/cm_btree.c b/src/WINNT/afsd/cm_btree.c
index a212df8b6..c7bc2b7e7 100644
--- a/src/WINNT/afsd/cm_btree.c
+++ b/src/WINNT/afsd/cm_btree.c
@@ -2336,11 +2336,11 @@ cm_BPlusDirEnumBulkStat(cm_scache_t *dscp, cm_direnum_t *enump, cm_user_t *userp
 long 
 cm_BPlusDirNextEnumEntry(cm_direnum_t *enump, cm_direnum_entry_t **entrypp)
 {	
-    if (enump == NULL || entrypp == NULL || enump->next > enump->count) {
+    if (enump == NULL || entrypp == NULL || enump->next >= enump->count) {
 	if (entrypp)
 	    *entrypp = NULL;
 	osi_Log0(afsd_logp, "cm_BPlusDirNextEnumEntry invalid input");
-	return CM_ERROR_INVAL;			      \
+	return CM_ERROR_INVAL;
     }
 
     *entrypp = &enump->entry[enump->next++];