[svn-r6632] Purpose:

    Bug fix

Description:
    This fixes a bug in the low-level metadata caching code in the library
which could possibly lose metadata during file I/O when a lot of objects are
inserted into a group.

    This also fixes a couple of (similar) fencepost bugs in the B-tree
deletion code.

Solution:
    For the metadata bug - call the low-level driver's 'write' routine instead
of H5FD_write.

    For the B-tree bug - include the correct number of keys.

Platforms tested:
    FreeBSD 4.8 (sleipnir) w/C++
    Linux 2.4 (burrwhite) w/FORTRAN
    Solaris 2.7 (arabica) w/FORTRAN
    IRIX64 6.5 (modi4) w/FORTRAN & parallel

    (h5committest is still not working for me on burrwhite)

Misc. update:

1 files changed, 2 insertions, 2 deletions

diff --git a/src/H5B.c b/src/H5B.c
index d298bc8..d761084 100644
--- a/src/H5B.c
+++ b/src/H5B.c
@@ -1738,7 +1738,7 @@ H5B_remove_helper(H5F_t *f, hid_t dxpl_id, haddr_t addr, const H5B_class_t *type
 	HDmemmove(bt->child,
 		  bt->child+1,
 		  bt->nchildren * sizeof(haddr_t));
-	for (i=0; i<bt->nchildren; i++) {
+	for (i=0; i<=bt->nchildren; i++) {
 	    bt->key[i].dirty = bt->key[i+1].dirty;
 	    if (bt->key[i+1].nkey) {
 		bt->key[i].nkey = bt->native + i*type->sizeof_nkey;
@@ -1787,7 +1787,7 @@ H5B_remove_helper(H5F_t *f, hid_t dxpl_id, haddr_t addr, const H5B_class_t *type
 	HDmemmove(bt->child+idx,
 		  bt->child+idx+1,
 		  (bt->nchildren-idx) * sizeof(haddr_t));
-	for (i=idx; i<bt->nchildren; i++) {
+	for (i=idx; i<=bt->nchildren; i++) {
 	    bt->key[i].dirty = bt->key[i+1].dirty;
 	    if (bt->key[i+1].nkey) {
 		bt->key[i].nkey = bt->native + i*type->sizeof_nkey;