[svn-r5498] Purpose:

    Code Bug Fix

Description:
    Under certain [obscure] circumstances, an object header would get paged out
    of the metadata cache, and when it was accessed again and brought back into
    the cache, and immediately had additional metadata added to it (an
    attribute, usually, or perhaps adding an object to a group), and needed to
    be extended with a continuation message, but there was no room in any
    existing object header chunks for the continuation message and an existing
    object header message needed to be moved to the new object header chunk (I
    told you it was obscure :-), the object header message moved to the new
    chunk (not the new metadata being added) would get corrupted. *whew* :-)

Solution:
    Actually copy the "raw" object header message information of the object
    header message being moved to the new chunk, instead of relying on the
    "native" object header message information being re-encoded when the object
    header is flushed.  This is because when an object header is paged out of
    the metadata cache and subsequently brought back in, the "native"
    information pointer in memory is reset to NULL and only the "raw"
    information exists.

Platforms tested:
    Solaris 2.7 (arabica) & FreeBSD 4.5 (sleipnir)

1 files changed, 2 insertions, 0 deletions

diff --git a/src/H5O.c b/src/H5O.c
index 5db9d21..6750ca7 100644
--- a/src/H5O.c
+++ b/src/H5O.c
@@ -1890,6 +1890,8 @@ H5O_alloc_new_chunk(H5F_t *f, H5O_t *oh, size_t size)
 	oh->mesg[i].chunkno = oh->mesg[found_other].chunkno;
 
 	oh->mesg[found_other].dirty = TRUE;
+        /* Copy the message to the new location */
+        HDmemcpy(p+H5O_SIZEOF_MSGHDR(f),oh->mesg[found_other].raw,oh->mesg[found_other].raw_size);
 	oh->mesg[found_other].raw = p + H5O_SIZEOF_MSGHDR(f);
 	oh->mesg[found_other].chunkno = chunkno;
 	p += H5O_SIZEOF_MSGHDR(f) + oh->mesg[found_other].raw_size;