diff options
| author | Raymond Lu <songyulu@hdfgroup.org> | 2009-06-26 20:24:42 (GMT) |
|---|---|---|
| committer | Raymond Lu <songyulu@hdfgroup.org> | 2009-06-26 20:24:42 (GMT) |
| commit | c0f0e7c4198de81d1d19c3a2e1a973380409737f (patch) | |
| tree | ee9923a72e2a529c377553a004f8b7983ca39743 /src/H5Aint.c | |
| parent | 4af1a11dcf7050adb19503f716fb4baee2bc7970 (diff) | |
| download | hdf5-c0f0e7c4198de81d1d19c3a2e1a973380409737f.zip hdf5-c0f0e7c4198de81d1d19c3a2e1a973380409737f.tar.gz hdf5-c0f0e7c4198de81d1d19c3a2e1a973380409737f.tar.bz2 | |
[svn-r17124] 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.
I tested it on jam only but have tested the same change for the trunk on jam, smirom,
and linew.
Diffstat (limited to 'src/H5Aint.c')
| -rw-r--r-- | src/H5Aint.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/H5Aint.c b/src/H5Aint.c index 9267c1b..4899a1c 100644 --- a/src/H5Aint.c +++ b/src/H5Aint.c @@ -1023,9 +1023,6 @@ H5A_attr_copy_file(const H5A_t *attr_src, H5F_t *file_dst, hbool_t *recompute_si if(H5A_set_version(file_dst, attr_dst) < 0) HGOTO_ERROR(H5E_ATTR, H5E_CANTSET, NULL, "unable to update attribute version") - /* Indicate that the fill values aren't to be written out */ - attr_dst->shared->initialized = TRUE; - /* Set return value */ ret_value = attr_dst; |