[svn-r8844] Purpose:

    Bug fix

Description:
    The "shared" raw B-tree node can get freed before all the B-tree nodes
had been flushed out to disk and released by the cache.

Solution:
    Implement a simple reference counting wrapper for objects in the library
and use it to hold the shared raw B-tree nodes so they aren't freed before all
references to them in memory are released.

Platforms tested:
    Solaris 2.7 (arabica)
    FreeBSD 4.10 (sleipnir)
    IRIX64 6.5 (modei4)

1 files changed, 4 insertions, 8 deletions

diff --git a/src/H5F.c b/src/H5F.c
index 86f9019..de16b40 100644
--- a/src/H5F.c
+++ b/src/H5F.c
@@ -1490,10 +1490,6 @@ H5F_new(H5F_file_t *shared, hid_t fcpl_id, hid_t fapl_id)
         /* Create the file's "open object" information */
         if(H5FO_create(f)<0)
 	    HGOTO_ERROR(H5E_FILE, H5E_CANTINIT, NULL, "unable to create open object TBBT")
-
-        /* Create information needed for group nodes */
-        if(H5G_node_init(f)<0)
-	    HGOTO_ERROR(H5E_FILE, H5E_CANTINIT, NULL, "unable to create group node info")
     } /* end else */
     
     f->shared->nrefs++;
@@ -4472,7 +4468,7 @@ done:
 
 
 /*-------------------------------------------------------------------------
- * Function:	H5F_raw_page
+ * Function:	H5F_rc_page
  *
  * Purpose:	Replaced a macro to retrieve the raw B-tree page value
  *              now that the generic properties are being used to store
@@ -4491,15 +4487,15 @@ done:
  *
  *-------------------------------------------------------------------------
  */
-void *H5F_raw_page(const H5F_t *f)
+H5RC_t *H5F_rc_page(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOFUNC here to avoid performance issues */
-    FUNC_ENTER_NOAPI_NOINIT_NOFUNC(H5F_raw_page)
+    FUNC_ENTER_NOAPI_NOINIT_NOFUNC(H5F_rc_page)
 
     assert(f);
     assert(f->shared);
 
-    FUNC_LEAVE_NOAPI(f->shared->raw_page)
+    FUNC_LEAVE_NOAPI(f->shared->rc_page)
 } /* end H5F_raw_page() */