diff options
author | Quincey Koziol <koziol@hdfgroup.org> | 2016-01-03 14:25:06 (GMT) |
---|---|---|
committer | Quincey Koziol <koziol@hdfgroup.org> | 2016-01-03 14:25:06 (GMT) |
commit | fc758c5cbe1018d11c7f97a162e80fbddda66789 (patch) | |
tree | 89b8ef1f526f729a8802cc9200279d91cd736447 | |
parent | f56120fdc30ea5a10d6c2b3e42d272d800fc8231 (diff) | |
download | hdf5-fc758c5cbe1018d11c7f97a162e80fbddda66789.zip hdf5-fc758c5cbe1018d11c7f97a162e80fbddda66789.tar.gz hdf5-fc758c5cbe1018d11c7f97a162e80fbddda66789.tar.bz2 |
[svn-r28774] Description:
Bring r28773 from trunk to revise_chunks branch:
Clear memory for shared message info when "deferring" and using memory
checker, so valgrind doesn't whine.
Tested on:
MacOSX/64 10.11.2 (amazon) w/serial & parallel
(h5committest not required on this branch)
-rw-r--r-- | src/H5SM.c | 14 |
1 files changed, 13 insertions, 1 deletions
@@ -1375,11 +1375,23 @@ H5SM_write_mesg(H5F_t *f, hid_t dxpl_id, H5O_t *open_oh, } /* end else */ } /* end else */ - if(found) + if(found) { /* If the message was found, it's shared in the heap (now). Set up a * shared message so we can mark it as shared. */ shared.type = H5O_SHARE_TYPE_SOHM; + +#ifdef H5_USING_MEMCHECKER + /* Reset the shared message payload if deferring. This doesn't matter + * in the long run since the payload will get overwritten when the + * non-deferred call to this routine occurs, but it stops memory + * checkers like valgrind from whining when the partially initialized + * shared message is serialized. -QAK + */ + if(defer) + HDmemset(&shared.u, 0, sizeof(shared.u)); +#endif /* H5_USING_MEMCHECKER */ + } /* end if */ else { htri_t share_in_ohdr; /* Whether the new message can be shared in another object's header */ |