diff options
| author | Raymond Lu <songyulu@hdfgroup.org> | 2009-06-26 19:41:32 (GMT) |
|---|---|---|
| committer | Raymond Lu <songyulu@hdfgroup.org> | 2009-06-26 19:41:32 (GMT) |
| commit | d8c0ebd42824f24ab457c1b98c90d0531f899103 (patch) | |
| tree | e11a6f171dc2f4a893c5067d093434ee46ef3aef /src/H5Oattr.c | |
| parent | 084ab0cbde453e8f5847abbb06bc94cfe6f3e657 (diff) | |
| download | hdf5-d8c0ebd42824f24ab457c1b98c90d0531f899103.zip hdf5-d8c0ebd42824f24ab457c1b98c90d0531f899103.tar.gz hdf5-d8c0ebd42824f24ab457c1b98c90d0531f899103.tar.bz2 | |
[svn-r17119] Bug fix #1513. Reading an empty attribute caused seg fault. The flag "initialized" in
the attribute structure wan't set correctly. It caused some confusion in H5A_read. This
flag was actually redundant because the library can alwasy check if the data buffer is
present. To fix it, I removed the "initialized" flag in the attribute structure and let
H5A_read check the data buffer.
Tested on jam, smirom, and linex.
Diffstat (limited to 'src/H5Oattr.c')
| -rw-r--r-- | src/H5Oattr.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/H5Oattr.c b/src/H5Oattr.c index 27b9d5e..1e85c69 100644 --- a/src/H5Oattr.c +++ b/src/H5Oattr.c @@ -226,9 +226,6 @@ H5O_attr_decode(H5F_t *f, hid_t dxpl_id, H5O_t *open_oh, unsigned UNUSED mesg_fl HDmemcpy(attr->shared->data, p, attr->shared->data_size); } /* end if */ - /* Indicate that the fill values aren't to be written out */ - attr->shared->initialized = 1; - /* Increment the reference count for this object header message in cache(compact storage) or for the object from dense storage. */ attr->shared->nrefs++; @@ -805,9 +802,6 @@ H5O_attr_debug(H5F_t *f, hid_t dxpl_id, const void *_mesg, FILE * stream, int in "Character Set of Name:", s); HDfprintf(stream, "%*s%-*s %t\n", indent, "", fwidth, - "Initialized:", - mesg->shared->initialized); - HDfprintf(stream, "%*s%-*s %t\n", indent, "", fwidth, "Object opened:", mesg->obj_opened); HDfprintf(stream, "%*s%-*s %a\n", indent, "", fwidth, |